Skip to content

Commit 25f6ab6

Browse files
nashifAnas Nashif
authored and
Anas Nashif
committed
Revert "sanitycheck: Default to using Ninja"
This reverts commit 0e6689d. Parallel builds are broken due to a mix of Make/Ninja and the job server not being operational. See ninja-build/ninja#1139 Signed-off-by: Anas Nashif <[email protected]>
1 parent 3a6d72e commit 25f6ab6

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

scripts/sanitycheck

+6-11
Original file line numberDiff line numberDiff line change
@@ -830,14 +830,14 @@ class MakeGenerator:
830830
if not "native_posix" in args:
831831
ldflags="-Wl,--fatal-warnings"
832832

833-
if options.make:
834-
generator = "Unix Makefiles"
835-
generator_cmd = "$(MAKE)"
836-
verb = "VERBOSE=1" if VERBOSE else "VERBOSE=0"
837-
else:
833+
if options.ninja:
838834
generator = "Ninja"
839835
generator_cmd = "ninja"
840836
verb = "-v" if VERBOSE else ""
837+
else:
838+
generator = "Unix Makefiles"
839+
generator_cmd = "$(MAKE)"
840+
verb = "VERBOSE=1" if VERBOSE else "VERBOSE=0"
841841

842842
return MakeGenerator.MAKE_RULE_TMPL.format(
843843
generator=generator,
@@ -2099,12 +2099,7 @@ def parse_arguments():
20992099

21002100
parser.add_argument(
21012101
"-N", "--ninja", action="store_true",
2102-
help="Use the Ninja generator with CMake. "
2103-
"This option is deprecated as Ninja is now the default.")
2104-
2105-
parser.add_argument(
2106-
"-M", "--make", action="store_true",
2107-
help="Use the 'Unix Makefiles' generator with CMake")
2102+
help="Use the Ninja generator with CMake")
21082103

21092104
parser.add_argument(
21102105
"-y", "--dry-run", action="store_true",

0 commit comments

Comments
 (0)