[PATCH v3 0/3] support pedantic in developer mode
- From
Carlo Marcelo Arenas Belón <carenas@gmail.com>
- Date
- Sep 3, 2021, 17:02 UTC
- Message-ID
- <20210903170232.57646-1-carenas@gmail.com>
- In-Reply-To
- <20210901091941.34886-1-carenas@gmail.com>
This series enables pedantic mode for building when DEVELOPER=1 is used and as an alternative to only enabling it in one CI job, that was merged to "seen" as part of cb/ci-build-pedantic.
The second patch is really an independent prerequisite to ensure that it doesn't break the build for Windows and is the minimal change possible.
Additional changes needed for the git-for-windows/git fork main to be posted independently.
It merges and builds successfully all the way to "seen" IF the known problem reported earlier[1] and expected as part of a reroll of jh/builtin-fsmonitor is merged first.
[1] https://lore.kernel.org/git/20210809063004.73736-3-carenas@gmail.com/
Carlo Marcelo Arenas Belón (2): win32: allow building with pedantic mode enabled developer: enable pedantic by default
Ævar Arnfjörð Bjarmason (1): gettext: remove optional non-standard parens in N_() definition
Makefile | 22 ++-------------------- compat/nedmalloc/nedmalloc.c | 2 +- compat/win32/lazyload.h | 2 +- config.mak.dev | 19 +++++++++++-------- gettext.h | 24 ------------------------ git-compat-util.h | 4 ---- 6 files changed, 15 insertions(+), 58 deletions(-)
-- v3 - replace the first patch with an even better worded one from Ævar - include minor changes needed for Windows - version check new flags to avoid risk of breaking old compilers - drop alternative v2 - enable pedantic globally instead of single job as suggested by Phillip - propose an alternative solution for USE_PARENS_AROUND_GETTEXT_N v1 - create job to check for pedantic compilation (only in Linux, using Fedora)
2.33.0.481.g26d3bed244