From e1385296c4ab4c7ee0a809676635b52d1df23b87 Mon Sep 17 00:00:00 2001 From: Andrew Grieve Date: Mon, 13 May 2024 03:11:00 +0000 Subject: [PATCH] Add "ninja -t targets" to the command allowlist It's used here: https://source.chromium.org/chromium/chromium/src/+/main:build/android/list_java_targets.py;drc=911128ac37a470797e6e24251598e4d0f3d3c21c;l=82 Change-Id: I31c687d36393a36b3b7d75b4668720214011aa1f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5527347 Auto-Submit: Andrew Grieve Commit-Queue: Fumitoshi Ukai Reviewed-by: Fumitoshi Ukai --- ninja.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ninja.py b/ninja.py index e0b2f0c4f..2b296ece4 100755 --- a/ninja.py +++ b/ninja.py @@ -42,7 +42,7 @@ def checkOutdir(ninja_args): out_dir = ninja_args[i + 1] elif arg.startswith("-C"): out_dir = arg[2:] - if tool in ["list", "commands", "inputs"]: + if tool in ["list", "commands", "inputs", "targets"]: # These tools are just inspect ninja rules and not modify out dir. # TODO: b/339320220 - implement these in siso return