Bug 282986 – dev-python/imaging-1.1.6-r1 failing during install

• Status: RESOLVED FIXED
• Product: Gentoo/Alt

[Description Michael Yang 2009-08-27 22:40:57 UTC]

trying to emerge dev-python/imaging-1.1.6-r1.  did not have this problem with dev-python/imaging-1.1.6

Reproducible: Always

Steps to Reproduce:
1. emerge =dev-python/imaging-1.1.6-r1



Expected Results:  
successful emerge

 * Installation of dev-python/imaging-1.1.6-r1 with Python 2.6...
-------------------------------------------------------
You should not use ${EPREFIX} with helpers.
  --> /home/michael/20090824-FreeBSD-i386/usr/include/python2.6
-------------------------------------------------------
-------------------------------------------------------
You should not use ${EPREFIX} with helpers.
  --> /home/michael/20090824-FreeBSD-i386/usr/include/python2.6
-------------------------------------------------------
 * ERROR: dev-python/imaging-1.1.6-r1 failed:
 *   Installation failed with Python 2.6 in install_headers() function

[Comment 1 Fabian Groffen 2009-09-01 19:36:59 UTC]

fixed in the eclass, should be solved now, thanks

[Comment 2 Jeremy Olexa (darkside) (RETIRED) 2009-09-01 20:57:25 UTC]

(In reply to comment #1)
> fixed in the eclass, should be solved now, thanks
> 

No, I have a feeling you should revert this eclass commit. See what I did here to work around a similar issue:

http://overlays.gentoo.org/proj/alt/changeset/50259

We can't go around adding $EPREFIX and yanking it out of eclasses on a whim... I'm pretty sure you just broke dev-python/sip --incdir=$(python_get_includedir) (maybe others).

[Comment 3 Jeremy Olexa (darkside) (RETIRED) 2009-09-01 20:59:04 UTC]

Also, it is horrible inconsistent to have python_get_includedir() return non EPREFIX path, and python_get_libdir() to return the EPREFIX path..

[Comment 4 Fabian Groffen 2009-09-02 06:23:10 UTC]

you're absolutely right sir.

python eclass used not to return EPREFIX, but after last total change, I decided it should, well, it's a mess.  The correct approach would be to let it return the prefix, so I'll revert right now!

[Comment 5 Fabian Groffen 2009-09-02 06:29:15 UTC]

ok, just fixed the ebuild this time.

[Comment 6 Jeremy Olexa (darkside) (RETIRED) 2009-09-02 13:23:44 UTC]

thanks.