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

Issue 163583003: Introduce the chroot_cmd flag that contains the command line prefix for running in a chrooted envir… (Closed)

Created:
6 years, 10 months ago by Alexander Potapenko
Modified:
6 years, 10 months ago
Reviewers:
piman
CC:
chromium-reviews
Visibility:
Public.

Description

Introduce the chroot_cmd flag that contains the command line prefix for running in a chrooted environment. Right now when supplying a 32-bit sysroot via the 'sysroot' GYP flag on a 64-bit host arch, pkg-config (and pkg-config-wrapper) fail to generate the correct 32-bit paths within that sysroot. The solution is to chroot into that directory and run pkg-config from there. This is intended to work with schroot scripts like 'precise32' generated when running build/install-chroot.sh BUG=324207 R=piman@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=251353

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M build/common.gypi View 2 chunks +2 lines, -0 lines 0 comments Download
M build/linux/system.gyp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
Alexander Potapenko
Please take a look
6 years, 10 months ago (2014-02-13 15:04:30 UTC) #1
piman
On 2014/02/13 15:04:30, Alexander Potapenko wrote: > Please take a look Why does sysroot= not ...
6 years, 10 months ago (2014-02-13 19:00:27 UTC) #2
piman
LGTM anyway for latency round trip reasons, 'cause I don't feel strongly about this if ...
6 years, 10 months ago (2014-02-13 19:02:26 UTC) #3
Alexander Potapenko
The sysroot flag makes the buildsystem run the 64-bit pkg-config on a 32-bit chroot, but ...
6 years, 10 months ago (2014-02-13 19:30:35 UTC) #4
Alexander Potapenko
For the record, here's the example: $ pkg-config --cflags glib-2.0 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include $ PKG_CONFIG_SYSROOT_DIR=/var/lib/chroot/precise32bit pkg-config ...
6 years, 10 months ago (2014-02-14 07:50:48 UTC) #5
Alexander Potapenko
Committed patchset #1 manually as r251353 (presubmit successful).
6 years, 10 months ago (2014-02-14 16:58:48 UTC) #6
piman
On Thu, Feb 13, 2014 at 11:50 PM, <glider@chromium.org> wrote: > For the record, here's ...
6 years, 10 months ago (2014-02-14 19:35:24 UTC) #7
Alexander Potapenko
6 years, 10 months ago (2014-02-14 19:45:55 UTC) #8
Yet it won't replace x86_64 with i386.
On Feb 14, 2014 11:35 PM, "Antoine Labour" <piman@chromium.org> wrote:

>
>
>
> On Thu, Feb 13, 2014 at 11:50 PM, <glider@chromium.org> wrote:
>
>> For the record, here's the example:
>>
>> $  pkg-config --cflags glib-2.0
>> -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
>>
>> $ PKG_CONFIG_SYSROOT_DIR=/var/lib/chroot/precise32bit pkg-config --cflags
>> glib-2.0
>> -I/var/lib/chroot/precise32bit/usr/include/glib-2.0
>> -I/var/lib/chroot/precise32bit/usr/lib/x86_64-linux-gnu/glib-2.0/include
>>
>
> pkg-config-wrapper will rewrite the -I lines so that they're relative to
> the sysroot.
>
>
>>
>> Committing the patch now.
>>
>> https://codereview.chromium.org/163583003/
>>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698