]> git.ipfire.org Git - thirdparty/git.git/commit
config.mak.dev: fix typo when enabling -Wpedantic
authorTaylor Blau <me@ttaylorr.com>
Fri, 5 Jul 2024 18:51:09 +0000 (14:51 -0400)
committerJunio C Hamano <gitster@pobox.com>
Sat, 6 Jul 2024 22:10:29 +0000 (15:10 -0700)
commitdf327298664ec9290d71c6086339823017f2caee
tree85aabfa55f6662ee470239adb2c2579fe80778b0
parentdbecc617f75eea6ee5d05823a38b91223ff22b11
config.mak.dev: fix typo when enabling -Wpedantic

In ebd2e4a13a (Makefile: restrict -Wpedantic and -Wno-pedantic-ms-format
better, 2021-09-28), we tightened our Makefile's behavior to only enable
-Wpedantic when compiling with either gcc5/clang4 or greater as older
compiler versions did not have support for -Wpedantic.

Commit ebd2e4a13a was looking for either "gcc5" or "clang4" to appear in
the COMPILER_FEATURES variable, combining the two "$(filter ...)"
searches with an "$(or ...)".

But ebd2e4a13a has a typo where instead of writing:

    ifneq ($(or ($filter ...),$(filter ...)),)

we wrote:

    ifneq (($or ($filter ...),$(filter ...)),)

Causing our Makefile (when invoked with DEVELOPER=1, and a sufficiently
recent compiler version) to barf:

    $ make DEVELOPER=1
    config.mak.dev:13: extraneous text after 'ifneq' directive
    [...]

Correctly combine the results of the two "$(filter ...)" operations by
using "$(or ...)", not "$or".

Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
config.mak.dev