Fix base_nocompile_tests dependency

Fix base_nocompile_tests dependency

nocompile_tests wasn't aware header file dependencies that included by
a .nc file. That makes a test failure sticky.

This CL makes nocompile_driver.py to emit a depfile to let ninja know
the implicit dependencies.

Committed: https://crrev.com/b5f8f4bca759e2535439edb022cc1f3e0b5a897d
Cr-Commit-Position: refs/heads/master@{#375504}

[tzik, 2016-02-15 04:40:49]

tzik@chromium.org changed reviewers:
+ dcheng@chromium.org, thakis@chromium.org

[tzik, 2016-02-15 04:40:50]

PTAL

[Nico, 2016-02-15 12:28:53]

lgtm

https://codereview.chromium.org/1698763002/diff/20001/tools/nocompile_driver.py
File tools/nocompile_driver.py (right):

https://codereview.chromium.org/1698763002/diff/20001/tools/nocompile_driver....
tools/nocompile_driver.py:197: 'cmdline': The exectued command line.
typo exectued->executed (not in your code, but since you're touching this file
maybe you can fix this quickly)

[Nico, 2016-02-15 12:29:58]

https://codereview.chromium.org/1698763002/diff/20001/tools/nocompile_driver.py
File tools/nocompile_driver.py (right):

https://codereview.chromium.org/1698763002/diff/20001/tools/nocompile_driver....
tools/nocompile_driver.py:438: cflags + ' -MMD -MF %s.d -MT %s' %
(resultfile_path, resultfile_path),
(note to future self: the -MT flag is needed because clang is passed -o
/dev/null and we manually write something to resultfile_path and since
resultfile_path is the rule's output as far as ninja is concerned, this must
match)

[tzik, 2016-02-16 01:25:12]

https://codereview.chromium.org/1698763002/diff/20001/tools/nocompile_driver.py
File tools/nocompile_driver.py (right):

https://codereview.chromium.org/1698763002/diff/20001/tools/nocompile_driver....
tools/nocompile_driver.py:197: 'cmdline': The exectued command line.
On 2016/02/15 12:28:53, Nico wrote:
> typo exectued->executed (not in your code, but since you're touching this file
> maybe you can fix this quickly)

Done.

[tzik, 2016-02-16 01:25:18]

The CQ bit was checked by tzik@chromium.org

[tzik, 2016-02-16 01:25:19]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1698763002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2016-02-16 01:25:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1698763002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1698763002/40001

[commit-bot: I haz the power, 2016-02-16 02:34:53]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-16 22:51:07]

Description was changed from

==========
Fix base_nocompile_tests dependency

nocompile_tests wasn't aware header file dependencies that included by
a .nc file. That makes a test failure sticky.

This CL makes nocompile_driver.py to emit a depfile to let ninja know
the implicit dependencies.
==========

to

==========
Fix base_nocompile_tests dependency

nocompile_tests wasn't aware header file dependencies that included by
a .nc file. That makes a test failure sticky.

This CL makes nocompile_driver.py to emit a depfile to let ninja know
the implicit dependencies.

Committed: https://crrev.com/b5f8f4bca759e2535439edb022cc1f3e0b5a897d
Cr-Commit-Position: refs/heads/master@{#375504}
==========

[commit-bot: I haz the power, 2016-02-16 22:51:08]

Patchset 3 (id:??) landed as
https://crrev.com/b5f8f4bca759e2535439edb022cc1f3e0b5a897d
Cr-Commit-Position: refs/heads/master@{#375504}