diff options
author | Dan Willemsen <dwillemsen@google.com> | 2016-09-29 20:09:47 -0700 |
---|---|---|
committer | Dan Willemsen <dwillemsen@google.com> | 2016-09-29 22:21:10 -0700 |
commit | f87d49e41a5dd57733d02f3990c91dc38e557dad (patch) | |
tree | 9533b73a9ec811257ed79f12467e50d83df0d2fd /symtab.h | |
parent | 5e45e973c38c92c42cc86aa5dafeca13e6823b5f (diff) | |
download | android_build_kati-f87d49e41a5dd57733d02f3990c91dc38e557dad.tar.gz android_build_kati-f87d49e41a5dd57733d02f3990c91dc38e557dad.tar.bz2 android_build_kati-f87d49e41a5dd57733d02f3990c91dc38e557dad.zip |
Support marking variables as readonly
When the magic variable .KATI_READONLY is set to a variable name, any
further attempts to modify the named variable will result in an error.
FOO := bar
.KATI_READONLY := FOO
FOO := baz # Error!
This is useful to make some global configuration readonly so that
another makefile cannot change it. In Android, we emulated this by
backing up some global configuration before including the Android.mk
files, then comparing the current values to the backed up values after
they've been included. But this means we don't know the location that
modified the variable, just that something did. And it's not perfect,
since the backup can also be changed.
Something similar to this could be implemented with `override`, but then
setting the variable silently fails, and it still could be overriden
with another override.
Diffstat (limited to 'symtab.h')
-rw-r--r-- | symtab.h | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -56,7 +56,7 @@ class Symbol { bool IsValid() const { return v_ >= 0; } Var* GetGlobalVar() const; - void SetGlobalVar(Var* v, bool is_override = false) const; + void SetGlobalVar(Var* v, bool is_override = false, bool* readonly = nullptr) const; private: explicit Symbol(int v); |