Use a stamp file when symlinking in GN work around mtime issues

Use a stamp file when symlinking in GN work around mtime issues

Ninja follows a symlink when stat'ing, so you can't use a symlink as an
output and have dirty checking work properly.
https://github.com/ninja-build/ninja/issues/1186

BUG=646165
Committed: https://crrev.com/fef37c4393d059c9125353928de7e49d4ea04ab4
Cr-Commit-Position: refs/heads/master@{#419156}

[agrieve, 2016-09-16 02:01:25]

The CQ bit was checked by agrieve@chromium.org to run a CQ dry run

[agrieve, 2016-09-16 02:01:37]

Description was changed from

==========
Use a stamp file when symlinking in GN work around mtime issues

Ninja follows a symlink when stat'ing, so you can't use a symlink as an
output and have dirty checking work properly.
https://github.com/ninja-build/ninja/issues/1186

BUG=646165
==========

to

==========
Use a stamp file when symlinking in GN work around mtime issues

Ninja follows a symlink when stat'ing, so you can't use a symlink as an
output and have dirty checking work properly.
https://github.com/ninja-build/ninja/issues/1186

BUG=646165
==========

[agrieve, 2016-09-16 02:01:37]

agrieve@chromium.org changed reviewers:
+ dpranke@chromium.org

[agrieve, 2016-09-16 02:01:42]

On 2016/09/16 02:01:37, agrieve wrote:
> mailto:agrieve@chromium.org changed reviewers:
> + mailto:dpranke@chromium.org

[commit-bot: I haz the power, 2016-09-16 02:01:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 02:14:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-16 02:14:08]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Dirk Pranke, 2016-09-16 03:25:22]

lgtm

[agrieve, 2016-09-16 13:03:57]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-09-16 13:04:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-16 13:37:34]

Description was changed from

==========
Use a stamp file when symlinking in GN work around mtime issues

Ninja follows a symlink when stat'ing, so you can't use a symlink as an
output and have dirty checking work properly.
https://github.com/ninja-build/ninja/issues/1186

BUG=646165
==========

to

==========
Use a stamp file when symlinking in GN work around mtime issues

Ninja follows a symlink when stat'ing, so you can't use a symlink as an
output and have dirty checking work properly.
https://github.com/ninja-build/ninja/issues/1186

BUG=646165
==========

[commit-bot: I haz the power, 2016-09-16 13:37:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-16 13:40:00]

Description was changed from

==========
Use a stamp file when symlinking in GN work around mtime issues

Ninja follows a symlink when stat'ing, so you can't use a symlink as an
output and have dirty checking work properly.
https://github.com/ninja-build/ninja/issues/1186

BUG=646165
==========

to

==========
Use a stamp file when symlinking in GN work around mtime issues

Ninja follows a symlink when stat'ing, so you can't use a symlink as an
output and have dirty checking work properly.
https://github.com/ninja-build/ninja/issues/1186

BUG=646165

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

[commit-bot: I haz the power, 2016-09-16 13:40:01]

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

[afakhry, 2016-09-16 16:49:19]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2348493005/ by afakhry@chromium.org.

The reason for reverting is: Breaks BuildPackages due to GN error
(https://bugs.chromium.org/p/chromium/issues/detail?id=647706).

BUG=647706.