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

Issue 4516002: Fix pkg-config-wrapper to work with chromiumos chroots (Closed)

Created:
10 years, 1 month ago by piman
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews
Visibility:
Public.

Description

Fix pkg-config-wrapper to work with chromiumos chroots Some pkg-config packages in chromium os (e.g. gtk) end up hard-coding libdir to be absolute (rather than relative to prefix), so it breaks builds from outside the chroot. This works around it. BUG=none TEST=build with sysroot=/path/to/chromiumos/chroot/build/x86-generic Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=65580

Patch Set 1 #

Patch Set 2 : Strip prefix instead of hacking libdir etc. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -15 lines) Patch
M build/linux/pkg-config-wrapper View 1 1 chunk +13 lines, -4 lines 0 comments Download
M build/linux/rewrite_dirs.py View 3 chunks +15 lines, -11 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
piman
10 years, 1 month ago (2010-11-05 02:09:28 UTC) #1
piman
ping ? Should I find another reviewer ? Antoine On Thu, Nov 4, 2010 at ...
10 years, 1 month ago (2010-11-07 18:49:34 UTC) #2
Evan Martin
10 years, 1 month ago (2010-11-09 06:34:57 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698