Hook up .d files and outputs to grit

Hook up .d files and outputs to grit

Pull grit 171 to get the new deps fixes required to implement this.

This removes the call to grit for querying inputs completely, and enables depfile writing to enable automatic rebuilds when things are out-of-date.

Adds a new outputs variable that bypasses querying grit for outputs. This will assert that the given files exist in grit. This version allows outputs to be empty in which case it will revert to the old dynamic behavior. I'm going to land this with just the worst cases fixed to make landing easier. Then followup with converting the rest of the grit targets and removing the dynamic option.

R=viettrungluu@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284705

[brettw, 2014-07-21 17:22:00]

This requires a grit roll first.

[viettrungluu, 2014-07-21 19:43:18]

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
File build/secondary/tools/grit/stamp_grit_sources.py (right):

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:1: # Copyright 2014 The
Chromium Authors. All rights reserved.
Please run pylint on this script.

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:13: #    stamp_grit_sources.py
<directory> <stamp-file> <.d-file>
It seems to me that this isn't very grit-specific at all, and would probably be
useful for various other things (written in python, for example) as well.

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:20: for root, dirs, filenames
in os.walk(grit_root_dir):
It's conventional to use _ for any variable you're going to ignore (dirs in this
case).

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:37: if len(sys.argv) != 4:
Please put all of this in a main() function (or Main(), if you want; our style
is somewhat inconsistent on this).

[brettw, 2014-07-21 20:11:19]

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
File build/secondary/tools/grit/stamp_grit_sources.py (right):

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:1: # Copyright 2014 The
Chromium Authors. All rights reserved.
On 2014/07/21 19:43:17, viettrungluu wrote:
> Please run pylint on this script.

Done.

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:13: #    stamp_grit_sources.py
<directory> <stamp-file> <.d-file>
Yes, I'm going to keep that in mind, but I don't want tp preemptively add this
to the build directory with no other users.

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:20: for root, dirs, filenames
in os.walk(grit_root_dir):
On 2014/07/21 19:43:17, viettrungluu wrote:
> It's conventional to use _ for any variable you're going to ignore (dirs in
this
> case).

Done.

https://codereview.chromium.org/407653003/diff/40001/build/secondary/tools/gr...
build/secondary/tools/grit/stamp_grit_sources.py:37: if len(sys.argv) != 4:
On 2014/07/21 19:43:17, viettrungluu wrote:
> Please put all of this in a main() function (or Main(), if you want; our style
> is somewhat inconsistent on this).

Done.

[viettrungluu, 2014-07-21 20:22:21]

lgtm

[brettw, 2014-07-22 16:37:33]

Committed patchset #4 manually as r284705.