| Summary: | Missing prefix in /usr/lib/portage/bin/phase-helpers.sh | ||
|---|---|---|---|
| Product: | Portage Development | Reporter: | Thierry Schuepbach <schoopy> |
| Component: | Core - Ebuild Support | Assignee: | Portage team <dev-portage> |
| Status: | RESOLVED NEEDINFO | ||
| Severity: | normal | CC: | prefix |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: | Patch to correct missing prefix | ||
Created attachment 363262 [details, diff]
Patch to correct missing prefix
@prefix: Could you comment on this? depends on CONF_PREFIX, if it never includes EPREFIX, then the patch seems fine Could you give me instructions how to reproduce this? The code seems to include EPREFIX in CONF_PREFIX, or assume it is (--exec-prefix=/--prefix= arguments), so if CONF_PREFIX doesn't include EPREFIX (hence you having to use ED) then it's most likely a problem of the configure script passing non-prefixed arguments. That said, I think the patch is wrong. |
Within einstall function in file /usr/lib/portage/bin/phase-helpers.sh, there should be a ${ED} instead of the actual ${D}. The line unfortunately changes between portage projects. Look in the code below on the second line. Hence prefix is not correctly being used. Nevertheless I believe this part of the code is hardly being used as it deals specific ebuils passing --bindir. if [ -n "${CONF_LIBDIR}" ] && [ "${CONF_PREFIX:+set}" = set ]; then EI_DESTLIBDIR="${D}/${CONF_PREFIX}/${CONF_LIBDIR}" EI_DESTLIBDIR="$(__strip_duplicate_slashes "${EI_DESTLIBDIR}")" LOCAL_EXTRA_EINSTALL="libdir=${EI_DESTLIBDIR} ${LOCAL_EXTRA_EINSTALL}" unset EI_DESTLIBDIR fi