Chromium Code Reviews
Help | Chromium Project | Sign in
(35)

Issue 11343015: Linux: add option to link libpci directly instead of using dlopen. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 5 months ago by Paweł Hajdan Jr.
Modified:
1 year, 5 months ago
Reviewers:
Zhenyao Mo, jam, piman
CC:
chromium-reviews_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium
Visibility:
Public.

Description

Linux: add option to link libpci directly instead of using dlopen.

This is for Linux distros, so they can detect more breakages
at compile time and using automated tools.

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=165482

Patch Set 1 #

Patch Set 2 : install-build-deps.sh #

Total comments: 3

Patch Set 3 : fixes #

Total comments: 2

Patch Set 4 : share more code #

Patch Set 5 : rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -49 lines) Lint Patch
M content/content_gpu.gypi View 3 2 chunks +20 lines, -0 lines 0 comments ? errors Download
M content/gpu/gpu_info_collector_linux.cc View 1 2 3 5 chunks +34 lines, -49 lines 0 comments 5 errors Download
Commit:

Messages

Total messages: 15
Paweł Hajdan Jr.
Please review: zmo: everything piman: OWNERS
1 year, 5 months ago #1
piman
https://codereview.chromium.org/11343015/diff/2001/content/content_gpu.gypi File content/content_gpu.gypi (right): https://codereview.chromium.org/11343015/diff/2001/content/content_gpu.gypi#newcode122 content/content_gpu.gypi:122: '<!@(pkg-config --cflags libpci)', This won't work if you have ...
1 year, 5 months ago #2
Paweł Hajdan Jr.
PTAL
1 year, 5 months ago #3
piman
LGTM for my parts, but please have zmo vet this.
1 year, 5 months ago #4
Zhenyao Mo
LGTM https://codereview.chromium.org/11343015/diff/8001/content/gpu/gpu_info_collector_linux.cc File content/gpu/gpu_info_collector_linux.cc (right): https://codereview.chromium.org/11343015/diff/8001/content/gpu/gpu_info_collector_linux.cc#newcode77 content/gpu/gpu_info_collector_linux.cc:77: PciInterface* interface = new struct PciInterface; Nit: this ...
1 year, 5 months ago #5
Paweł Hajdan Jr.
http://codereview.chromium.org/11343015/diff/8001/content/gpu/gpu_info_collector_linux.cc File content/gpu/gpu_info_collector_linux.cc (right): http://codereview.chromium.org/11343015/diff/8001/content/gpu/gpu_info_collector_linux.cc#newcode77 content/gpu/gpu_info_collector_linux.cc:77: PciInterface* interface = new struct PciInterface; On 2012/10/29 21:39:08, ...
1 year, 5 months ago #6
Zhenyao Mo
LGTM
1 year, 5 months ago #7
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/phajdan.jr@chromium.org/11343015/9008
1 year, 5 months ago #8
I haz the power (commit-bot)
Failed to apply patch for build/install-build-deps.sh: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
1 year, 5 months ago #9
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/phajdan.jr@chromium.org/11343015/12002
1 year, 5 months ago #10
I haz the power (commit-bot)
Change committed as 165482
1 year, 5 months ago #11
jam
I just synced and now I get: ../../content/gpu/gpu_info_collector_linux.cc:13:21: error: pci/pci.h: No such file or directory ...
1 year, 5 months ago #12
piman
On Thu, Nov 1, 2012 at 2:04 PM, <jam@chromium.org> wrote: > I just synced and ...
1 year, 5 months ago #13
Paweł Hajdan Jr.
PSA sent. John, sorry about the surprise build failure, now the change is announced. Please ...
1 year, 5 months ago #14
jam
1 year, 5 months ago #15
that worked,t hanks

On Thu, Nov 1, 2012 at 2:12 PM, <phajdan.jr@chromium.org> wrote:

> PSA sent. John, sorry about the surprise build failure, now the change is
> announced. Please let me know if problems continue after running
> install-build-deps.sh
>
>
https://chromiumcodereview.**appspot.com/11343015/<https://chromiumcodereview...
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1275:d14800f88434