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

Issue 10537170: Print full path in diagnostic messages building with ninja on windows. (Closed)

Created:
8 years, 6 months ago by Vitaly Buka (NO REVIEWS)
Modified:
8 years, 6 months ago
Reviewers:
scottmg
CC:
chromium-reviews, gyp-developer_googlegroups.com
Visibility:
Public.

Description

Print full path in diagnostic messages building with ninja on windows. This allows to click on messages in visual studio to open file with error or warning. By default all path is relative to src/out/Debug but visual studio search file relative to location project. Putting project into deeper directory to match default relative path is an option, but in this case you can't use 'Show All Files' in solution explorer to browse source tree. BUG=none TEST=none Committed: https://code.google.com/p/gyp/source/detail?r=1417

Patch Set 1 : #

Patch Set 2 : Print full path in diagnostic messages building with ninja on windows. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M pylib/gyp/generator/ninja.py View 1 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
scottmg
That's a good idea, it's annoying have the (wrong) out\Debug-relative path. We should probably put ...
8 years, 6 months ago (2012-06-15 21:15:02 UTC) #1
Vitaly Buka (NO REVIEWS)
Print full path in diagnostic messages building with ninja on windows. This allows to click ...
8 years, 6 months ago (2012-06-16 04:05:44 UTC) #2
Vitaly Buka (NO REVIEWS)
Done. On 2012/06/15 21:15:02, scottmg wrote: > That's a good idea, it's annoying have the ...
8 years, 6 months ago (2012-06-16 04:06:40 UTC) #3
scottmg
lgtm
8 years, 6 months ago (2012-06-18 20:15:37 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/10537170/11001
8 years, 6 months ago (2012-06-18 20:17:27 UTC) #5
commit-bot: I haz the power
Failed to apply patch for pylib/gyp/generator/ninja.py: While running patch -p0 --forward --force; A pylib A ...
8 years, 6 months ago (2012-06-18 20:17:35 UTC) #6
scottmg
On 2012/06/18 20:17:35, I haz the power (commit-bot) wrote: > Failed to apply patch for ...
8 years, 6 months ago (2012-06-18 20:18:48 UTC) #7
M-A Ruel
On 2012/06/18 20:18:48, scottmg wrote: > On 2012/06/18 20:17:35, I haz the power (commit-bot) wrote: ...
8 years, 6 months ago (2012-06-18 20:24:55 UTC) #8
Vitaly Buka (NO REVIEWS)
8 years, 6 months ago (2012-06-18 20:33:23 UTC) #9
How can I commit this CL?
I have no commiter access to this repo.

Powered by Google App Engine
This is Rietveld 408576698