commit 71ee193e797b49a11775fa5f8cbfc00673e5fb0e
author Paul Mathieu <paulmathieu@google.com> Thu May 27 19:06:29 2021 -0700
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Sep 13 15:55:20 2021 +0000
tree 3312427cfa0b43a8e76b29f2e161ec17037ffd32
parent 9c771de68ffba834cb94c5eb4dc7edc2cb347f0f

python_runner: fix target finding at root

At the root level, where the default dummy toolchain is used, the stamp
tool is just 'stamp', so <TARGET_FILE()> wasn't working for them.

Now does.

No-Docs-Update-Reason: bugfix
Change-Id: I35c9f5ee084bbefafa6f9a5fb6112e2d097767ee
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/47400
Commit-Queue: Paul Mathieu <paulmathieu@google.com>
Reviewed-by: Wyatt Hepler <hepler@google.com>

pw_build/py/pw_build/python_runner.py

diff --git a/pw_build/py/pw_build/python_runner.py b/pw_build/py/pw_build/python_runner.py
index 5c38717..7a21350 100755
--- a/pw_build/py/pw_build/python_runner.py
+++ b/pw_build/py/pw_build/python_runner.py
@@ -254,7 +254,9 @@
 
     # Older versions of GN used a .stamp file to signal completion of a target.
     stamp_dir = target.out_dir.relative_to(paths.build).as_posix()
-    stamp_tool = f'{target.toolchain_name()}_stamp'
+    stamp_tool = 'stamp'
+    if target.toolchain_name() != '':
+        stamp_tool = f'{target.toolchain_name()}_stamp'
     stamp_statement = f'build {stamp_dir}/{target.name}.stamp: {stamp_tool} '
 
     # Newer GN uses a phony Ninja target to signal completion of a target.