Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 269780 - gnome-extra/gnome-games should allow choice between sdl and gstreamer
Summary: gnome-extra/gnome-games should allow choice between sdl and gstreamer
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords: Inclusion
Depends on: 271076
Blocks:
  Show dependency tree
 
Reported: 2009-05-14 06:08 UTC by Matthew Turnbull
Modified: 2009-05-31 12:47 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Modified gnome-games-2.26 ebuild with gstreamer and sdl use flags (gnome-games-2.26.1-r1.ebuild,4.11 KB, text/plain)
2009-05-14 13:43 UTC, Matthew Turnbull
Details
Patch for previous ebuild, plus an additional warning about disabled sound support (gnome-games.patch,1.70 KB, patch)
2009-05-14 15:10 UTC, Matthew Turnbull
Details | Diff
Make gstreamer the default, fall back on sdl-mixer (gnome-games-2.26.2-sdl-mixer-fallback.patch,1.78 KB, patch)
2009-05-24 15:55 UTC, Arun Raghavan (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Turnbull 2009-05-14 06:08:06 UTC
I've been trying to rid my system of gstreamer, and one of my last remaining dependencies is from gnome-games.

A cursory glance shows that gnome-games can be built against sdl (default) or gstreamer (which the ebuild explicitly selects).

Unless there is a reason for this, I think that gstreamer support should be enabled via use flag, and should fall back on sdl when disabled.

Reproducible: Always
Comment 1 Gilles Dartiguelongue (RETIRED) gentoo-dev 2009-05-14 08:22:51 UTC
care enough to try and submit and patch for the ebuild ?
Comment 2 Matthew Turnbull 2009-05-14 13:43:08 UTC
Created attachment 191263 [details]
Modified gnome-games-2.26 ebuild with gstreamer and sdl use flags

I made sdl the dominant option because it's the default for the release and IMHO it sounds better than gstreamer when they're backed by pulseaudio.
Comment 3 Rémi Cardona (RETIRED) gentoo-dev 2009-05-14 13:51:40 UTC
FTR, I've checked upstream's code and libcanberra is a possibility at least in git master...

Maybe a patch that's worth backporting?
Comment 4 Rémi Cardona (RETIRED) gentoo-dev 2009-05-14 13:52:20 UTC
@Matthew, please attach diffs instead of full ebuilds. Makes reviewing much easier.

Thanks
Comment 5 Matthew Turnbull 2009-05-14 15:10:08 UTC
Created attachment 191273 [details, diff]
Patch for previous ebuild, plus an additional warning about disabled sound support

Sorry, I couldn't remember which format was preferred.

I think it would be nice to backport the libcanberra changes, but it looks like its still undergoing implementation changes. By the time it stabilizes, GNOME 2.28 might be close enough to just wait for it.
Comment 6 Arun Raghavan (RETIRED) gentoo-dev 2009-05-24 15:55:32 UTC
Created attachment 192310 [details, diff]
Make gstreamer the default, fall back on sdl-mixer

Thanks for the patch! I'm attaching a slightly different patch that makes gstreamer the default option and falls back on sdl-mixer otherwise.

I don't want to drop keywords on sh support for this patch, so I've filed another one (blocking this bug) requesting sh support in sdl-mixer first.
Comment 7 Gilles Dartiguelongue (RETIRED) gentoo-dev 2009-05-28 11:44:46 UTC
Arun, your patch looks good. One thing though, I think users would like to know in advance if they'll get sound or not. If they're not in front of their computer at the moment it gets printed it's not important if it's before or after building anyway.
Comment 8 Gilles Dartiguelongue (RETIRED) gentoo-dev 2009-05-31 12:47:19 UTC
Included in 2.26.2-r1. Thanks for reporting and the patch.