<?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>207298</bug_id>
          
          <creation_ts>2008-01-24 16:39 0000</creation_ts>
          <short_desc>x11-drivers/nvidia-drivers relying upon internal portage function</short_desc>
          <delta_ts>2008-02-15 15:13:24 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>Ebuilds</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>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>peper@gentoo.org</reporter>
          <assigned_to>cardoe@gentoo.org</assigned_to>
          <cc>amd64@gentoo.org</cc>
    
    <cc>dev-portage@gentoo.org</cc>
    
    <cc>eradicator@gentoo.org</cc>

      

      
          <long_desc isprivate="0">
            <who>peper@gentoo.org</who>
            <bug_when>2008-01-24 16:39:30 0000</bug_when>
            <thetext>nvidia-drivers-* :

src_install() {
	local MLTEST=$(type dyn_unpack)
(...)

	if [[ &quot;${MLTEST/set_abi}&quot; == &quot;${MLTEST}&quot; ]] &amp;&amp; has_multilib_profile ; then

the dyn_unpack thing was added here:
http://sources.gentoo.org/viewcvs.py/gentoo-x86/media-video/nvidia-glx/nvidia-glx-1.0.6629-r3.ebuild?hideattic=0&amp;r1=1.4&amp;r2=1.5

the MLTEST at all seems to first apear in this ebuild:
http://sources.gentoo.org/viewcvs.py/gentoo-x86/media-video/nvidia-glx/nvidia-glx-1.0.6629-r2.ebuild?hideattic=0&amp;rev=1.1&amp;view=markup

I /think/ this was added to check for some particular abi-related functionality in portage. Whatever it was seems to be outdated for quite some time as I didn&apos;t find &quot;set_abi&quot; in any dyn_unpack implementation since portage repo was moved to svn, but I didn&apos;t really check that carefully as it&apos;s wrong anyway.

Please KILL.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>cardoe@gentoo.org</who>
            <bug_when>2008-01-24 16:49:18 0000</bug_when>
            <thetext>Until someone from the CC list can explain what was intended by it, I&apos;m not removing it.

@eradicator: You added this code. Can you provide some details?

@amd64: This code is for you guys... Can you explain?</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>eradicator@gentoo.org</who>
            <bug_when>2008-01-24 17:39:49 0000</bug_when>
            <thetext>IIRC, MLTEST was used in a portage branch maybe in 2003? 2004? when I was employed by GenUX to develop eselect-compiler, eselect-binutils, and add better multilib abstraction to portage.

In short, GenUX pushed for things that were impractical, they went belly-up, and all that code entered bit-rot.

I think it&apos;s safe just to
-        local MLTEST=$(type dyn_unpack)
-        if [[ &quot;${MLTEST/set_abi}&quot; == &quot;${MLTEST}&quot; ]] &amp;&amp; has_multilib_profile ;
+        if has_multilib_profile ;

</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>halcy0n@gentoo.org</who>
            <bug_when>2008-01-25 16:29:03 0000</bug_when>
            <thetext>Unless the maintainers do not understand why this is a problem, or they are being uncooperative, there is no reason to add QA to this bug.  Please don&apos;t add us back unless one of the above conditions is met.  If we don&apos;t follow those guidelines, we might as well CC QA on every single bug in bugzilla.

Thanks</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>wolf31o2@gentoo.org</who>
            <bug_when>2008-01-25 18:59:15 0000</bug_when>
            <thetext>I am no longer maintaining nvidia-*drivers in an way.  Doug has been doing an excellent job and I didn&apos;t have much to do.  Removing myself from CC.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>cardoe@gentoo.org</who>
            <bug_when>2008-02-15 15:13:24 0000</bug_when>
            <thetext>fixed in 169.09-r1, the change will propagate to other versions as well.</thetext>
          </long_desc>
      
    </bug>

</bugzilla>