Bug 704719 - Split pygame MIDI support into subpackage
Summary: Split pygame MIDI support into subpackage
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: pygame
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jan Kaluža
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-05-14 11:02 UTC by Daniel Drake
Modified: 2011-05-14 11:27 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-05-14 11:27:28 UTC
Type: ---


Attachments (Terms of Use)
spec file patch (3.59 KB, patch)
2011-05-14 11:03 UTC, Daniel Drake
no flags Details | Diff

Description Daniel Drake 2011-05-14 11:02:01 UTC
pygame-1.9 adds MIDI support via the pygame.midi module which was enabled bug #585526.

The midi module depends on portmidi which is Java-based.

This causes some headaches for OLPC (and any small Sugar installation) as it means that Java and java's dependency chain is added to our software image, which eats a big chunk of precious disk space (XO-1 laptop only has 1gb disk).

Please can you split the midi part into a subpackage, so that we can easily exclude it from OLPC's builds and not get the whole Java invasion?

I'll attach a patch that does this. I've checked that it does this safely - pygame does list midi support as an optional dependency, and if portmidi is not available at build time the end result is simply that pypm.so is not installed.

Comment 1 Daniel Drake 2011-05-14 11:03:31 UTC
Created attachment 498919 [details]
spec file patch

Comment 2 Daniel Drake 2011-05-14 11:27:28 UTC
ah, I just looked closer and portmidi is just a small library and doesn't actually require java. a small mistake in the spec file pulls in java for no reason. I'll make the trivial fix to portmidi, meaning that theres no need for changes to pygame.


Note You need to log in before you can comment on or make changes to this bug.