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

Issue 11862016: Make use_system_mesa switch work: make symlinks (Closed)

Created:
7 years, 11 months ago by Paweł Hajdan Jr.
Modified:
7 years, 11 months ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews
Visibility:
Public.

Description

Make use_system_mesa switch work: make symlinks for nacl untrusted build. BUG=161389 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=178028

Patch Set 1 #

Total comments: 4

Patch Set 2 : fixes #

Patch Set 3 : symlink.py #

Patch Set 4 : no debugging code #

Unified diffs Side-by-side diffs Delta from patch set Stats (+95 lines, -5 lines) Patch
A build/symlink.py View 1 2 3 1 chunk +44 lines, -0 lines 0 comments Download
M third_party/khronos/khronos.gyp View 1 2 3 chunks +51 lines, -5 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Paweł Hajdan Jr.
7 years, 11 months ago (2013-01-17 20:35:47 UTC) #1
Mark Mentovai
https://codereview.chromium.org/11862016/diff/1/third_party/khronos/khronos.gyp File third_party/khronos/khronos.gyp (right): https://codereview.chromium.org/11862016/diff/1/third_party/khronos/khronos.gyp#newcode57 third_party/khronos/khronos.gyp:57: '<(khronos_headers_symlink_target)/.dummy_gles2', What’s your theory here? You just want to ...
7 years, 11 months ago (2013-01-18 17:38:45 UTC) #2
Paweł Hajdan Jr.
https://codereview.chromium.org/11862016/diff/1/third_party/khronos/khronos.gyp File third_party/khronos/khronos.gyp (right): https://codereview.chromium.org/11862016/diff/1/third_party/khronos/khronos.gyp#newcode57 third_party/khronos/khronos.gyp:57: '<(khronos_headers_symlink_target)/.dummy_gles2', On 2013/01/18 17:38:45, Mark Mentovai wrote: > What’s ...
7 years, 11 months ago (2013-01-18 18:36:43 UTC) #3
Mark Mentovai
The best I can come up with is to have the .gyp file itself be ...
7 years, 11 months ago (2013-01-18 18:54:35 UTC) #4
Paweł Hajdan Jr.
On 2013/01/18 18:54:35, Mark Mentovai wrote: > The best I can come up with is ...
7 years, 11 months ago (2013-01-18 20:46:01 UTC) #5
Mark Mentovai
I think it’s worth it to avoid resymlinking each time. That’s bad for builds and ...
7 years, 11 months ago (2013-01-18 20:48:48 UTC) #6
Paweł Hajdan Jr.
Done. PTAL.
7 years, 11 months ago (2013-01-18 23:51:30 UTC) #7
Mark Mentovai
7 years, 11 months ago (2013-01-19 16:03:33 UTC) #8
LGTM

Powered by Google App Engine
This is Rietveld 408576698