diff options
| author | Jeroen van Rijn <Kelimion@users.noreply.github.com> | 2025-06-02 14:35:24 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-06-02 14:35:24 +0200 |
| commit | 2788cfaa0db9c88a887190e37b7703260d36f8e8 (patch) | |
| tree | ed9e23ef923831c5167a829a013c8e3ccd411263 /src/main.cpp | |
| parent | e0c9bcbb803c7662f444871847f2c09882dd4ec5 (diff) | |
| parent | 16b8da6a79dcfeb02c3e7b3ff47df0d56bf88f1d (diff) | |
Merge pull request #5255 from Feoramund/build-mode-test-all-packages
Let `-test-all-packages` work with `-build-mode:test`
Diffstat (limited to 'src/main.cpp')
| -rw-r--r-- | src/main.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/main.cpp b/src/main.cpp index b7c3e5abc..531bbe98e 100644 --- a/src/main.cpp +++ b/src/main.cpp @@ -596,7 +596,7 @@ gb_internal bool parse_build_flags(Array<String> args) { add_flag(&build_flags, BuildFlag_ObfuscateSourceCodeLocations, str_lit("obfuscate-source-code-locations"), BuildFlagParam_None, Command__does_build); add_flag(&build_flags, BuildFlag_Short, str_lit("short"), BuildFlagParam_None, Command_doc); - add_flag(&build_flags, BuildFlag_AllPackages, str_lit("all-packages"), BuildFlagParam_None, Command_doc | Command_test); + add_flag(&build_flags, BuildFlag_AllPackages, str_lit("all-packages"), BuildFlagParam_None, Command_doc | Command_test | Command_build); add_flag(&build_flags, BuildFlag_DocFormat, str_lit("doc-format"), BuildFlagParam_None, Command_doc); add_flag(&build_flags, BuildFlag_IgnoreWarnings, str_lit("ignore-warnings"), BuildFlagParam_None, Command_all); @@ -1733,6 +1733,12 @@ gb_internal bool parse_build_flags(Array<String> args) { bad_flags = true; } + + if ((build_context.command_kind & (Command_doc | Command_test)) == 0 && build_context.test_all_packages) { + gb_printf_err("`-test-all-packages` can only be used with `odin build -build-mode:test`, `odin test`, or `odin doc`.\n"); + bad_flags = true; + } + return !bad_flags; } |