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

Issue 7067039: Add a "dump-json" generator that dumps the dependency graph. (Closed)

Created:
9 years, 7 months ago by Evan Martin
Modified:
9 years, 7 months ago
Reviewers:
Mark Mentovai, Nico
CC:
gyp-developer_googlegroups.com
Visibility:
Public.

Description

Add a "dump-json" generator that dumps the dependency graph. - Add a generator-specific variable to turn off the automatic flattening of static library dependencies. - Add a new simple generator module that just dumps the graph as JSON.

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -1 line) Patch
M pylib/gyp/__init__.py View 1 chunk +2 lines, -0 lines 1 comment Download
A pylib/gyp/generator/dump-json.py View 1 chunk +56 lines, -0 lines 1 comment Download
M pylib/gyp/input.py View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Evan Martin
Initially I put all this graphviz code in here, but then after looking at some ...
9 years, 7 months ago (2011-05-24 21:20:44 UTC) #1
Mark Mentovai
LG otherwise http://codereview.chromium.org/7067039/diff/1/pylib/gyp/__init__.py File pylib/gyp/__init__.py (right): http://codereview.chromium.org/7067039/diff/1/pylib/gyp/__init__.py#newcode80 pylib/gyp/__init__.py:80: 'generator_wants_flattened_static_libraries': “flattened” is a bad name because ...
9 years, 7 months ago (2011-05-24 21:46:15 UTC) #2
Nico
I have a patch that adds a json output generator as well (for a completely ...
9 years, 7 months ago (2011-05-25 01:57:20 UTC) #3
Evan Martin
9 years, 7 months ago (2011-05-25 02:29:16 UTC) #4
On 2011/05/25 01:57:20, Nico wrote:
> I have a patch that adds a json output generator as well (for a completely
> different cause). Maybe this one could be called dump-dependency-json, since
> json is such a generic term?

Ack, sorry!  I will rename it (I already committed it, not sure why it didnt'
close)

Powered by Google App Engine
This is Rietveld 408576698