Summary: | einstall modifies EXTRA_EINSTALL | ||
---|---|---|---|
Product: | Portage Development | Reporter: | James C. Georgas <james.georgas> |
Component: | Core | Assignee: | Portage team <dev-portage> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Keywords: | InVCS |
Priority: | High | ||
Version: | 2.1 | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 181949 | ||
Attachments: |
preserve EXTRA_EINSTALL in einstall() function
updated patch to einstall() in ebuild.sh |
Description
James C. Georgas
2007-05-14 19:31:17 UTC
Created attachment 119249 [details, diff]
preserve EXTRA_EINSTALL in einstall() function
This patch preserves the contents of EXTRA_EINSTALL, by using the EI_DESTLIBDIR variable directly instead.
`einstall` shouldnt be used in the first place, but that's a different topic the usage should be changed to match EXTRA_ECONF usage in econf() ... in other words, introduce a local LOCAL_EXTRA_EINSTALL and utilize that Created attachment 119319 [details, diff]
updated patch to einstall() in ebuild.sh
This patch incorporates spanKY's suggestions to solve the problem.
thanks, ive merged that one Your welcome. I have another bug, similar to this one, reported as #178378, which I think got lost between the cracks last week. It started out as an eclass bug, but the problem also exists in ebuild.sh, so it might be relevant to core also. (In reply to comment #5) > I have another bug, similar to this one, reported as #178378, which I think got > lost between the cracks last week. It started out as an eclass bug, but the > problem also exists in ebuild.sh, so it might be relevant to core also. In svn r6558. This has been released in 2.1.2.8. |