<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "http://bugs.gentoo.org/bugzilla.dtd">

<bugzilla version="2.22.7"
          urlbase="http://bugs.gentoo.org/"
          maintainer="bugzilla@gentoo.org"
>

    <bug>
          <bug_id>203594</bug_id>
          
          <creation_ts>2007-12-28 17:40 0000</creation_ts>
          <short_desc>media-video/mplayer should only depend on x264-svn if both encode and x264 USE flags are enabled</short_desc>
          <delta_ts>2008-01-03 02:04:05 0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>Gentoo Linux</product>
          <component>Applications</component>
          <version>unspecified</version>
          <rep_platform>All</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          
          <priority>P2</priority>
          <bug_severity>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>hopeless@hamiltonshells.ca</reporter>
          <assigned_to>media-video@gentoo.org</assigned_to>
          

      

      
          <long_desc isprivate="0">
            <who>hopeless@hamiltonshells.ca</who>
            <bug_when>2007-12-28 17:40:12 0000</bug_when>
            <thetext>`emerge mplayer` pulls in media-libs/x264-svn if the &quot;x264&quot; USE flag is enabled, however it&apos;s not actually used if the &quot;encode&quot; USE flag is also enabled.

From the ebuild (mplayer-1.0_rc2_p24929);
        if use encode; then
                use aac || myconf=&quot;${myconf} --disable-faac&quot;
                use dv || myconf=&quot;${myconf} --disable-libdv&quot;
                use x264 || myconf=&quot;${myconf} --disable-x264&quot;
        else
                myconf=&quot;${myconf} --disable-mencoder --disable-libdv --disable-x264 \
                        --disable-faac&quot;
        fi

Interestingly, I have mplayer installed on two boxes, both with x264 in USE but only one with encode.
x264-svn is pulled in in both cases, but as you can see it&apos;s disabled if mencoder isn&apos;t built, however `ldd /usr/bin/mplayer` on the box with encode in USE shows that the mplayer binary is actually linked to libx264.so as well as mencoder, suggesting the possiblity that mplayer itself makes use of it even without mencoder.

I suggest either changing the ebuild to depend on x264-svn if the x264 USE flag is enabled irrespective of whether or not encode is also enabled, OR changing the depends so x264-svn is only pulled in if the encode flag is enabled as well as x264.

Reproducible: Always</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>hopeless@hamiltonshells.ca</who>
            <bug_when>2007-12-28 17:41:20 0000</bug_when>
            <thetext>That should read &quot;however it&apos;s not actually used UNLESS the &quot;encode&quot; USE flag is also enabled.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>jakub@gentoo.org</who>
            <bug_when>2007-12-28 18:21:28 0000</bug_when>
            <thetext>Reopen with a patch or something instead of this messy description.
</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>hopeless@hamiltonshells.ca</who>
            <bug_when>2007-12-28 22:34:03 0000</bug_when>
            <thetext>Fair enough, I&apos;ll attach a patch to mplayer-1.0_rc2_p24929.ebuild so it only pulls in x264-svn if the encode USE flag is also active.

Anyway, even if mplayer is compiled with libx264 support it still uses ffmpeg&apos;s ffh264 for playback by default.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>hopeless@hamiltonshells.ca</who>
            <bug_when>2007-12-28 22:35:09 0000</bug_when>
            <thetext>Created an attachment (id=139540)
patch to fix the x264svn RDEPEND

</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>jakub@gentoo.org</who>
            <bug_when>2007-12-28 22:40:00 0000</bug_when>
            <thetext>Thanks, much more clear now... :)</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>beandog@gentoo.org</who>
            <bug_when>2008-01-03 02:04:05 0000</bug_when>
            <thetext>Fixed in CVS, thanks</thetext>
          </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>139540</attachid>
            <date>2007-12-28 22:35 0000</date>
            <desc>patch to fix the x264svn RDEPEND</desc>
            <filename>mplayer_x264depends.patch</filename>
            <type>text/plain</type>
            <data encoding="base64">LS0tIG1wbGF5ZXItMS4wX3JjMl9wMjQ5MjkuZWJ1aWxkLm9yaWcJMjAwNy0xMi0yOCAyMjoyNDo0
Ni4wMDAwMDAwMDAgKzAwMDAKKysrIG1wbGF5ZXItMS4wX3JjMl9wMjQ5MjkuZWJ1aWxkCTIwMDct
MTItMjggMjI6MjU6MzYuMDAwMDAwMDAwICswMDAwCkBAIC02MSw2ICs2MSw3IEBACiAJCWFhYz8g
KCBtZWRpYS1saWJzL2ZhYWMgKQogCQltcDI/ICggbWVkaWEtc291bmQvdHdvbGFtZSApCiAJCW1w
Mz8gKCBtZWRpYS1zb3VuZC9sYW1lICkKKwkJeDI2ND8gKCBtZWRpYS1saWJzL3gyNjQtc3ZuICkK
IAkJKQogCWVzZD8gKCBtZWRpYS1zb3VuZC9lc291bmQgKQogCWVuY2E/ICggYXBwLWkxOG4vZW5j
YSApCkBAIC05NSw3ICs5Niw2IEBACiAJCW1lZGlhLWxpYnMvZm9udGNvbmZpZyApCiAJdmlkaXg/
ICggeDExLWxpYnMvbGliWHhmODZ2bQogCQkJIHgxMS1saWJzL2xpYlhleHQgKQotCXgyNjQ/ICgg
bWVkaWEtbGlicy94MjY0LXN2biApCiAJeGFuaW0/ICggbWVkaWEtdmlkZW8veGFuaW0gKQogCXhp
bmVyYW1hPyAoIHgxMS1saWJzL2xpYlhpbmVyYW1hCiAJCXgxMS1saWJzL2xpYlh4Zjg2dm0K
</data>        

          </attachment>
    </bug>

</bugzilla>