Bug 356923 – python.eclass should support prefixed path

• Status: RESOLVED WONTFIX
• Product: Gentoo/Alt

[Description Benda Xu 2011-03-01 04:23:46 UTC]

python_get_includedir and python_get_libdir does not repect ${EROOT}.

This causes problems for some ebuilds(esp. newly ecopied) in gentoo prefix.

[Comment 1 Jeremy Olexa (darkside) (RETIRED) 2011-03-01 04:38:22 UTC]

Likely "wontfix" - eclasses are not definable one way or another because of how functions are used. What helps one ebuild may very well break another one, etc.

[Comment 2 Benda Xu 2011-03-01 06:53:18 UTC]

Created attachment 264225 [details, diff]
patch that using EROOT

[Comment 3 Benda Xu 2011-03-01 06:57:01 UTC]

(In reply to comment #1)
> Likely "wontfix" - eclasses are not definable one way or another because of how
> functions are used. What helps one ebuild may very well break another one, etc.

in python.eclass, some functions are already using EROOT, and there is a helper function to determine EROOT (see the patch)

Maybe you are right. Then we need to maintain ebuild patches against non-prefixed python includedir and lib.

[Comment 4 Benda Xu 2011-03-01 06:59:49 UTC]

(In reply to comment #1)
> Likely "wontfix" - eclasses are not definable one way or another because of how
> functions are used. What helps one ebuild may very well break another one, etc.
> 

In gentoo-x86 ${EROOT} is defined as ${ROOT} which in turn is "/". The only scene of breaking another ebuild would be double prefix, with which we can unpatch and do that systematically with eclass.

[Comment 5 Fabian Groffen 2011-03-01 08:19:23 UTC]

I think this was/is by design, as ROOT isn't used either.  The eclass requires callers to prepend ROOT/EPREFIX when necessary, e.g. when not used with helpers.

[Comment 6 Benda Xu 2011-03-01 11:35:23 UTC]

Okay, I see. Close.