diff options
| author | Dan Willemsen <dwillemsen@google.com> | 2018-08-27 15:01:03 -0700 |
|---|---|---|
| committer | Dan Willemsen <dwillemsen@google.com> | 2018-08-27 15:01:03 -0700 |
| commit | d8aa39d1c31389fffee933f94057a1c9dc4160b2 (patch) | |
| tree | fd4378c9506ea68596c4dff370d8881a4b59f421 /ui/build/kati.go | |
| parent | 0d91f1727ef200e5235598f75c97c6bdab687d2d (diff) | |
| download | build_soong-d8aa39d1c31389fffee933f94057a1c9dc4160b2.tar.gz build_soong-d8aa39d1c31389fffee933f94057a1c9dc4160b2.tar.bz2 build_soong-d8aa39d1c31389fffee933f94057a1c9dc4160b2.zip | |
Support turning phony warnings into errors
These still default to warnings, but if a board decides to set
BOARD_BROKEN_PHONY_TARGETS := false, they'll turn into errors. More
likely I'll just be marking the broken targets as broken, then switching
the logic here to be like the dup rules warnings/errors.
Test: On a build with warnings, try <missing>, "false", and "true"
Change-Id: I041e1ff4618c4114ec43015b6c0ae1b49b36b6f8
Diffstat (limited to 'ui/build/kati.go')
| -rw-r--r-- | ui/build/kati.go | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/ui/build/kati.go b/ui/build/kati.go index de8dbf4b..b54872c1 100644 --- a/ui/build/kati.go +++ b/ui/build/kati.go @@ -93,6 +93,10 @@ func runKati(ctx Context, config Config) { args = append(args, "--werror_overriding_commands") } + if !config.BuildBrokenPhonyTargets() { + args = append(args, "--werror_real_to_phony", "--werror_phony_looks_real") + } + if !config.Environment().IsFalse("KATI_EMULATE_FIND") { args = append(args, "--use_find_emulator") } |