diff options
author | Rémi Verschelde <rverschelde@gmail.com> | 2021-10-15 21:59:11 +0200 |
---|---|---|
committer | Rémi Verschelde <rverschelde@gmail.com> | 2021-10-15 22:14:11 +0200 |
commit | c13348053129d4a356a0df1ada208809997799fe (patch) | |
tree | cf00036a5834e76eee0eda0be4e7ad8232544a61 /editor | |
parent | 08d1ce718d5d1845c163f016b4083210136c1248 (diff) |
SCons: List `.gen.cpp` sources explicitly to avoid globbing errors
Whenever we change the name (or remove) generated cpp files with the `.gen.cpp`
extension, users run into build issues when switching between branches (i.e.
switching before and after the name change/removal). This is because we glob
`*.cpp` so if a now-obsolete file from a previous build is present, we'll
include it too, potentially leading to bugs or compilation failure (due to
missing headers or invalid code).
So globbing patterns in `add_source_files` will now skip files ending with
`.gen.cpp`, which should instead be passed explicitly where they're used.
Diffstat (limited to 'editor')
-rw-r--r-- | editor/SCsub | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/editor/SCsub b/editor/SCsub index d149cc6273..e8d417319a 100644 --- a/editor/SCsub +++ b/editor/SCsub @@ -101,6 +101,7 @@ if env["tools"]: ) env.add_source_files(env.editor_sources, "*.cpp") + env.add_source_files(env.editor_sources, "register_exporters.gen.cpp") SConscript("debugger/SCsub") SConscript("fileserver/SCsub") |