Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(463)

Issue 6903153: Check for references to ${ROOT} in install_qa_checks. (Closed)

Created:
9 years, 7 months ago by davidjames
Modified:
9 years, 4 months ago
Reviewers:
zmedico, anush
CC:
chromium-os-reviews_chromium.org, zbehan, mtennant
Base URL:
git://git.overlays.gentoo.org/proj/portage.git@master
Visibility:
Public.

Description

Check for references to ${ROOT} in install_qa_checks. When ROOT != /, binaries that reference ROOT will load their dependencies from ROOT first rather than from the system-configured path. This is a problem because the ROOT will be / on the target system. Besides the above, this patch also fixes incorrect parsing of scanelf output, where we would treat the RPATHs returned by scanelf as the names of binaries. TEST=When "stricter" FEATURE is enabled, verify that emerge fails when an ebuild references broken rpaths referencing ROOT. When "stricter" FEATURE is not enabled, verify that such references are automatically fixed. Also verify that ebuilds with non-broken RPATHs (e.g. RPATHs referencing $ORIGIN/../lib) are not touched by the change. BUG=chromium-os:14271 Change-Id: I4f29cc4ea9195a1255f080284da1f676e4a2c26b R=anush@chromium.org,zmedico@gmail.com

Patch Set 1 #

Patch Set 2 : s/SYSROOT/ROOT/g per zmedico #

Patch Set 3 : Be safe with whitespace #

Patch Set 4 : Unused var #

Patch Set 5 : Typo fix #

Patch Set 6 : Final version merged #

Patch Set 7 : Same patch, rebased against cros-2.1.9 branch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -8 lines) Patch
M bin/misc-functions.sh View 1 2 3 4 5 6 2 chunks +26 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
davidjames
9 years, 7 months ago (2011-04-29 22:09:31 UTC) #1
zmedico
9 years, 7 months ago (2011-04-30 00:26:57 UTC) #2
I've merged this upstream:

http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=e483cb18...

I added this one line to make it behave more like before:

   vecho "Auto fixing rpaths for ${l%%:*}"

Powered by Google App Engine
This is Rietveld 408576698