diff options
author | Rémi Verschelde <rverschelde@gmail.com> | 2022-09-07 09:11:23 +0200 |
---|---|---|
committer | Rémi Verschelde <rverschelde@gmail.com> | 2022-09-07 09:11:23 +0200 |
commit | 94361494fd45cfd1c51f80765e83566f589c5642 (patch) | |
tree | 19f2765008088748d9040494737b3bec552e7f17 | |
parent | 647f345ffed8b06cdb2fa70ec4ffc575888ecfe3 (diff) | |
parent | 22381d9d074f77f4acb700c4d765e44e3f0727f7 (diff) |
Merge pull request #65440 from akien-mga/scons-compiledb-msvc
Fix compilation database generation with MSVC
-rw-r--r-- | SConstruct | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/SConstruct b/SConstruct index 5ad4b614ca..ce586010f4 100644 --- a/SConstruct +++ b/SConstruct @@ -438,19 +438,6 @@ if selected_platform in platform_list: ) env.SetOption("num_jobs", safer_cpu_count) - if env["compiledb"]: - # Generating the compilation DB (`compile_commands.json`) requires SCons 4.0.0 or later. - from SCons import __version__ as scons_raw_version - - scons_ver = env._get_major_minor_revision(scons_raw_version) - - if scons_ver < (4, 0, 0): - print("The `compiledb=yes` option requires SCons 4.0 or later, but your version is %s." % scons_raw_version) - Exit(255) - - env.Tool("compilation_db") - env.Alias("compiledb", env.CompilationDatabase()) - # 'dev' and 'production' are aliases to set default options if they haven't been set # manually by the user. if env["dev"]: @@ -840,6 +827,19 @@ if selected_platform in platform_list: env.vs_incs = [] env.vs_srcs = [] + if env["compiledb"]: + # Generating the compilation DB (`compile_commands.json`) requires SCons 4.0.0 or later. + from SCons import __version__ as scons_raw_version + + scons_ver = env._get_major_minor_revision(scons_raw_version) + + if scons_ver < (4, 0, 0): + print("The `compiledb=yes` option requires SCons 4.0 or later, but your version is %s." % scons_raw_version) + Exit(255) + + env.Tool("compilation_db") + env.Alias("compiledb", env.CompilationDatabase()) + Export("env") # Build subdirs, the build order is dependent on link order. |