884423 – install-qa-check.d/60pkgconfig: libdir check wrongly uses ${ED} instead of ${EROOT}?

Bug 884423 - install-qa-check.d/60pkgconfig: libdir check wrongly uses ${ED} instead of ${EROOT}?

Summary: install-qa-check.d/60pkgconfig: libdir check wrongly uses ${ED} instead of ${...

Status:	CONFIRMED

Alias:	None

Product:	Portage Development
Classification:	Unclassified
Component:	Core (show other bugs)
Hardware:	All Linux

Importance:	Normal normal
Assignee:	Portage team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2022-12-05 08:27 UTC by Sam James
Modified:	2022-12-05 08:27 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Sam James archtester

2022-12-05 08:27:30 UTC

[07:46:31]  <[Arfrever]> sam_: I also noticed that https://gitweb.gentoo.org/proj/portage.git/commit/?id=0ac65ddf7cff60a64730ca7c123f492fb68217a4 added:
[07:46:34]  <[Arfrever]> +            if [[ -d "${ED}"/usr/lib && -L "${ED}"/usr/lib ]] ; then
[07:46:36]  <[Arfrever]> +                # (Don't bother if /usr/lib is a symlink to /usr/lib64)
[07:46:39]  <[Arfrever]> +                continue
[07:47:28]  <[Arfrever]> Regardless of whether it is necessary, it is unlikely to have any effect with ${ED} being here.
[07:48:01]  <[Arfrever]> Probably ${EROOT} was intended.