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

Issue 9423022: mac ninja&make: Filter out useless 'This view is clipping its content' message. (Closed)

Created:
8 years, 10 months ago by Nico
Modified:
8 years, 10 months ago
Reviewers:
Robert Sesek
CC:
gyp-developer_googlegroups.com
Visibility:
Public.

Description

mac ninja&make: Filter out useless 'This view is clipping its content' message. BUG=none TEST=chromium build output is almost 200 lines shorter Committed: https://code.google.com/p/gyp/source/detail?r=1223

Patch Set 1 #

Patch Set 2 : #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -2 lines) Patch
M pylib/gyp/mac_tool.py View 1 2 chunks +14 lines, -2 lines 2 comments Download

Messages

Total messages: 6 (0 generated)
Nico
8 years, 10 months ago (2012-02-20 05:02:51 UTC) #1
Robert Sesek
Can we do this without the regexp but instead by passing flags to ibtool? I ...
8 years, 10 months ago (2012-02-20 15:08:39 UTC) #2
(unused - use chromium)
On 2012/02/20 15:08:39, rsesek wrote: > Can we do this without the regexp but instead ...
8 years, 10 months ago (2012-02-20 15:23:36 UTC) #3
Robert Sesek
https://chromiumcodereview.appspot.com/9423022/diff/2001/pylib/gyp/mac_tool.py File pylib/gyp/mac_tool.py (right): https://chromiumcodereview.appspot.com/9423022/diff/2001/pylib/gyp/mac_tool.py#newcode83 pylib/gyp/mac_tool.py:83: current_section_header = None Can there be multiple lines of ...
8 years, 10 months ago (2012-02-21 15:53:28 UTC) #4
Nico
https://chromiumcodereview.appspot.com/9423022/diff/2001/pylib/gyp/mac_tool.py File pylib/gyp/mac_tool.py (right): https://chromiumcodereview.appspot.com/9423022/diff/2001/pylib/gyp/mac_tool.py#newcode83 pylib/gyp/mac_tool.py:83: current_section_header = None On 2012/02/21 15:53:28, rsesek wrote: > ...
8 years, 10 months ago (2012-02-21 16:23:46 UTC) #5
Robert Sesek
8 years, 10 months ago (2012-02-21 20:20:52 UTC) #6
lgtm

Powered by Google App Engine
This is Rietveld 408576698