Skip to content

Commit 49f54c4

Browse files
avarttaylorr
authored andcommitted
Makefile: split off SPATCH_BATCH_SIZE comment from "cocci" heading
Split off the "; setting[...]" part of the comment added in In 960154b (coccicheck: optionally batch spatch invocations, 2019-05-06), and restore what we had before that, which was a comment indicating that variables for the "coccicheck" target were being set here. When 960154b amended the heading to discuss SPATCH_BATCH_SIZE it left no natural place to add a new comment about other flags that preceded it. As subsequent commits will add such comments we need to split the existing comment up. The wrapping for the "SPATCH_BATCH_SIZE" is now a bit odd, but minimizes the diff size. As a subsequent commit will remove that feature altogether this is worth it. Signed-off-by: Ævar Arnfjörð Bjarmason <[email protected]> Signed-off-by: Taylor Blau <[email protected]>
1 parent 09d9a69 commit 49f54c4

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

Makefile

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1294,10 +1294,11 @@ SP_EXTRA_FLAGS = -Wno-universal-initializer
12941294
SANITIZE_LEAK =
12951295
SANITIZE_ADDRESS =
12961296

1297-
# For the 'coccicheck' target; setting SPATCH_BATCH_SIZE higher will
1297+
# For the 'coccicheck' target
1298+
SPATCH_FLAGS = --all-includes
1299+
# Setting SPATCH_BATCH_SIZE higher will
12981300
# usually result in less CPU usage at the cost of higher peak memory.
12991301
# Setting it to 0 will feed all files in a single spatch invocation.
1300-
SPATCH_FLAGS = --all-includes
13011302
SPATCH_BATCH_SIZE = 1
13021303

13031304
# Rebuild 'coccicheck' if $(SPATCH), its flags etc. change

0 commit comments

Comments
 (0)