Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 505396 - media-video/handbrake USE=gstreamer should depend on media-plugins/gst-plugins-base:1.0
Summary: media-video/handbrake USE=gstreamer should depend on media-plugins/gst-plugin...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Tom Wijsman (TomWij) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-23 05:12 UTC by Abigail
Modified: 2014-03-26 15:40 UTC (History)
2 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Abigail 2014-03-23 05:12:38 UTC
When installing handbrake 0.9.9, gst-plugins-base-1.x and friends aren't pulled in if they aren't already installed. In the ebuild for handbrake, gst-plugins-base:1.0 is listed in RDEPEND.

Since handbrake would complain about missing plugins when using things such as the preview feature, I installed the 1.2.3 versions manually by issuing an "emerge -1ua gst-plugins-meta" since I already had the 0.10 version installed by other packages. This installed the 1.2.3 version in a new slot. Handbrake works fine afterwards.

Even though these packages are needed by handbrake, emerge wants to depclean the 1.2.3 gst-plugin packages.


emerge --info:

Portage 2.2.8-r1 (default/linux/amd64/13.0/desktop/kde, gcc-4.7.3, glibc-2.17, 3.4.60-gentoo x86_64)
=================================================================
System uname: Linux-3.4.60-gentoo-x86_64-Intel-R-_Core-TM-_i5_CPU_M_430_@_2.27GHz-with-gentoo-2.2
KiB Mem:     5909756 total,    133760 free
KiB Swap:    5242876 total,   4918968 free
Timestamp of tree: Sat, 22 Mar 2014 03:00:01 +0000
ld GNU ld (GNU Binutils) 2.23.2
app-shells/bash:          4.2_p45
dev-java/java-config:     2.2.0
dev-lang/python:          2.7.5-r3, 3.3.3
dev-util/cmake:           2.8.11.2
dev-util/pkgconfig:       0.28
sys-apps/baselayout:      2.2
sys-apps/openrc:          0.12.4
sys-apps/sandbox:         2.6-r1
sys-devel/autoconf:       2.13, 2.69
sys-devel/automake:       1.11.6, 1.12.6, 1.13.4
sys-devel/binutils:       2.23.2
sys-devel/gcc:            4.7.3-r1
sys-devel/gcc-config:     1.7.3
sys-devel/libtool:        2.4.2
sys-devel/make:           3.82-r4
sys-kernel/linux-headers: 3.9 (virtual/os-headers)
sys-libs/glibc:           2.17
Repositories: gentoo
ACCEPT_KEYWORDS="amd64"
ACCEPT_LICENSE="* -@EULA"
CBUILD="x86_64-pc-linux-gnu"
CFLAGS="-march=native -mmmx -msse -msse2 -msse3 -mssse3 -msse4.1 -msse4.2 -O2 -pipe -ggdb"
CHOST="x86_64-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/share/config /usr/share/gnupg/qualified.txt"
CONFIG_PROTECT_MASK="/etc/ca-certificates.conf /etc/env.d /etc/fonts/fonts.conf /etc/gconf /etc/gentoo-release /etc/revdep-rebuild /etc/sandbox.d /etc/terminfo"
CXXFLAGS="-march=native -mmmx -msse -msse2 -msse3 -mssse3 -msse4.1 -msse4.2 -O2 -pipe -ggdb"
DISTDIR="/usr/portage/distfiles"
FCFLAGS="-O2 -pipe"
FEATURES="assume-digests binpkg-logs candy config-protect-if-modified distlocks ebuild-locks fixlafiles merge-sync news parallel-fetch preserve-libs protect-owned sandbox sfperms splitdebug strict unknown-features-warn unmerge-logs unmerge-orphans userfetch userpriv usersandbox usersync"
FFLAGS="-O2 -pipe"
GENTOO_MIRRORS="http://distfiles.gentoo.org"
LANG="en_US.UTF-8"
LDFLAGS="-Wl,-O1 -Wl,--as-needed"
MAKEOPTS="-j5"
PKGDIR="/usr/portage/packages"
PORTAGE_CONFIGROOT="/"
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --omit-dir-times --compress --force --whole-file --delete --stats --human-readable --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY=""
USE="X a52 aac acl acpi alsa amd64 bash-completion berkdb bluray branding bzip2 cairo cd cdda cdr clamav cli consolekit cracklib crypt cups cxx dbus declarative dri dts dvd dvdr emboss encode exif fam ffmpeg firefox flac fortran gdbm geoip gif gphoto2 gpm iconv icu joystick jpeg kde kipi lcms ldap libnotify lm_sensors mad matroska mmx mng modules mp3 mp4 mpeg ncurses nls nptl ogg opengl openmp pam pango pcre pdf phonon plasma png policykit ppds qt3support qt4 readline samba scanner sdl session spell sse sse2 sse3 ssl ssse3 startup-notification svg syslog tcpd theora threads tiff truetype udev udisks unicode upower usb v4l video vim-syntax vorbis wxwidgets x264 xcb xcomposite xinerama xml xscreensaver xv xvid zlib" ABI_X86="64" ALSA_CARDS="ali5451 als4000 atiixp atiixp-modem bt87x ca0106 cmipci emu10k1x ens1370 ens1371 es1938 es1968 fm801 hda-intel intel8x0 intel8x0m maestro3 trident usb-audio via82xx via82xx-modem ymfpci" APACHE2_MODULES="authn_core authz_core socache_shmcb unixd actions alias auth_basic authn_alias authn_anon authn_dbm authn_default authn_file authz_dbm authz_default authz_groupfile authz_host authz_owner authz_user autoindex cache cgi cgid dav dav_fs dav_lock deflate dir disk_cache env expires ext_filter file_cache filter headers include info log_config logio mem_cache mime mime_magic negotiation rewrite setenvif speling status unique_id userdir usertrack vhost_alias" CALLIGRA_FEATURES="kexi words flow plan sheets stage tables krita karbon braindump author" CAMERAS="ptp2" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf superstar2 timing tsip tripmate tnt ublox ubx" INPUT_DEVICES="evdev synaptics" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LIBREOFFICE_EXTENSIONS="presenter-console presenter-minimizer" OFFICE_IMPLEMENTATION="libreoffice" PHP_TARGETS="php5-5" PYTHON_SINGLE_TARGET="python2_7" PYTHON_TARGETS="python2_7 python3_3" RUBY_TARGETS="ruby19 ruby20" SANE_BACKENDS="net hp" USERLAND="GNU" VIDEO_CARDS="radeon intel" XTABLES_ADDONS="quota2 psd pknock lscan length2 ipv4options ipset ipp2p iface geoip fuzzy condition tee tarpit sysrq steal rawnat logmark ipmark dhcpmac delude chaos account"
Unset:  CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS, SYNC, USE_PYTHON

Reproducible: Always

Steps to Reproduce:
1. On stable system (perhaps with gstreamer-0.10 and plugins already installed), add media-video/handbrake, media-video/ffmpeg-1.2.6, and virtual/ffmpeg-9 to package.accept_keywords
2. Add "media-video/handbrake gtk gstreamer" to package.use if the flags aren't already enabled.
3. emerge handbrake
4. Run ghb, load a source file (or DVD), click "Picture Settings", and play the preview video. Handbrake will complain about a missing gst-plugin and won't play the preview.
5. emerge -1ua gst-plugins-meta (or try gst-plugins-base)
6. Repeat step 4. Handbrake should play the preview without complaining about any missing plugins.
7. Run "emerge -p --depclean". Depclean wants to remove all the gst-plugins 1.2.3 packages (if nothing else but handbrake depends on them).
Actual Results:  
Portage doesn't install the gst-plugins-base:1.0 along with handbrake, even though it's listed in RDEPEND. Depclean wants to remove the gst-plugins 1.x packages (if nothing else but Handbrake depends on them).

Expected Results:  
Portage should install the gst-plugins 1.x along with handbrake if they aren't already installed. Depclean shouldn't remove them if handbrake is still installed.
Comment 1 Tom Wijsman (TomWij) (RETIRED) gentoo-dev 2014-03-24 12:41:55 UTC
(In reply to Abigail from comment #0)
> When installing handbrake 0.9.9, gst-plugins-base-1.x and friends aren't
> pulled in if they aren't already installed. In the ebuild for handbrake,
> gst-plugins-base:1.0 is listed in RDEPEND.

That is a contradiction; if it is listed there, it gets pulled. If it wouldn't be pulled, that would be a bug in Portage.
 
> Even though these packages are needed by handbrake, emerge wants to depclean
> the 1.2.3 gst-plugin packages.

Similarly, they wouldn't be depcleaned here then.

> 2. Add "media-video/handbrake gtk gstreamer" to package.use if the flags
> aren't already enabled.
> 3. emerge handbrake

This guarantees that, under a working Portage, gst-plugins-base:1.0 is pulled.

> 7. Run "emerge -p --depclean". Depclean wants to remove all the gst-plugins
> 1.2.3 packages (if nothing else but handbrake depends on them).

And this wouldn't happen.

> Actual Results:  
> Portage doesn't install the gst-plugins-base:1.0 along with handbrake, even
> though it's listed in RDEPEND. Depclean wants to remove the gst-plugins 1.x
> packages (if nothing else but Handbrake depends on them).

It should.
 
> Expected Results:  
> Portage should install the gst-plugins 1.x along with handbrake if they
> aren't already installed. Depclean shouldn't remove them if handbrake is
> still installed.

This is what is intended to happen, as well as happens for me. What am I missing? Is -base sufficient or should it be -meta? Do you see this as a Portage bug?
Comment 2 Tom Wijsman (TomWij) (RETIRED) gentoo-dev 2014-03-24 12:47:10 UTC
As far as I can see, only base is needed:

 $ grep gst doc/BUILD-Linux
   * libgstreamer0.10-dev (gui)
   * libgstreamer-plugins-base0.10-dev (gui)
     libgstreamer0.10-dev libgstreamer-plugins-base0.10-dev libappindicator-dev
   * gstreamer-devel (gui)
   * gstreamer-plugins-base-devel (gui)
     gstreamer-devel gstreamer-plugins-base-devel

And thus:

 $ grep -r 'libs/gst' handbrake-0.9.9.ebuild 
		media-libs/gstreamer:1.0
		media-libs/gst-plugins-base:1.0

Can you try again and correct me, yourself or confirm this is a Portage bug?
Comment 3 Abigail 2014-03-24 22:35:43 UTC
> > Expected Results:  
> > Portage should install the gst-plugins 1.x along with handbrake if they
> > aren't already installed. Depclean shouldn't remove them if handbrake is
> > still installed.
> 
> This is what is intended to happen, as well as happens for me. What am I
> missing? Is -base sufficient or should it be -meta? Do you see this as a
> Portage bug?

Doesn't seem to be a Portage bug. I've synced my tree again, and Portage didn't depclean gst-plugins-base:1.0 and other packages like gst-plugins-ffmpeg. But the handbrake ebuild seems to be missing dependencies. I can confirm that handbrake needs the following packages:

media-libs/gst-plugins-good:1.0
media-plugins/gst-plugins-x264:1.0 (pulls in media-libs/gst-plugins-ugly:1.0)
media-plugins/gst-plugins-a52dec:1.0

And maybe:

media-libs/gst-plugins-bad:1.0

What puzzles me is that although I did install the x264 plugin, handbrake still gives me a popup dialog that says that it's missing. But the activity log doesn't show any signs of GStreamer complaining, so I think I'm missing something. The preview window gives me a black screen, but sound plays fine. Handbrake did work fine when I updated all the packages contained in gst-plugins-meta:1.0, so I'm probably missing something that's listed in there.
Comment 4 Tom Wijsman (TomWij) (RETIRED) gentoo-dev 2014-03-26 15:40:55 UTC
Seems that is the case, Debian does this too:

pkg/linux/debian/control.quantal:Depends: ${shlibs:Depends}, ${misc:Depends}, gstreamer1.0-libav, gstreamer1.0-plugins-base, gstreamer1.0-plugins-good, gstreamer1.0-plugins-bad, gstreamer1.0-plugins-ugly, gstreamer1.0-pulseaudio
pkg/linux/debian/control.raring:Depends: ${shlibs:Depends}, ${misc:Depends}, gstreamer1.0-libav, gstreamer1.0-plugins-base, gstreamer1.0-plugins-good, gstreamer1.0-plugins-bad, gstreamer1.0-plugins-ugly, gstreamer1.0-pulseaudio

And this is confirmed; so, let's do it at Gentoo as well:

+               media-libs/gst-plugins-good:1.0
+               media-libs/gst-plugins-bad:1.0
+               media-libs/gst-plugins-ugly:1.0
+               media-plugins/gst-plugins-a52dec:1.0
+               media-plugins/gst-plugins-libav:1.0
+               media-plugins/gst-plugins-x264:1.0

+  26 Mar 2014; Tom Wijsman <TomWij@gentoo.org> +handbrake-0.9.9-r1.ebuild,
+  handbrake-9999.ebuild:
+  Revision bump, add more gstreamer dependencies to unbreak parts of the
+  application; fixes bug #505396, reported by Abigail.

Thank you very much for reporting