diff options
author | Matthew Pickering <matthewtpickering@gmail.com> | 2022-02-16 10:40:55 +0000 |
---|---|---|
committer | Marge Bot <ben+marge-bot@smart-cactus.org> | 2022-02-17 05:25:28 -0500 |
commit | 4e6c80197f1cc46dfdef0300de46847c7cfbdcb0 (patch) | |
tree | 72b3ddcfd121f161f5f611ac250941c1c58296d9 | |
parent | 4127e86dfdfcfce60e5029c716c98c244851779a (diff) | |
download | haskell-4e6c80197f1cc46dfdef0300de46847c7cfbdcb0.tar.gz |
Always define __GLASGOW_HASKELL_PATCHLEVEL1/2__ macros
As #21076 reports if you are using `-Wcpp-undef` then you get warnings
when using the `MIN_VERSION_GLASGOW_HASKELL` macro because
__GLASGOW_HASKELL_PATCHLEVEL2__ is very rarely explicitliy set (as
version numbers are not 4 components long).
This macro was introduced in 3549c952b535803270872adaf87262f2df0295a4
and it seems the bug has existed ever since.
Fixes #21076
-rw-r--r-- | m4/fp_setup_project_version.m4 | 4 | ||||
-rw-r--r-- | testsuite/tests/driver/all.T | 1 | ||||
-rw-r--r-- | testsuite/tests/driver/patch-level2.hs | 6 |
3 files changed, 11 insertions, 0 deletions
diff --git a/m4/fp_setup_project_version.m4 b/m4/fp_setup_project_version.m4 index 26df01865b..d51759add7 100644 --- a/m4/fp_setup_project_version.m4 +++ b/m4/fp_setup_project_version.m4 @@ -77,6 +77,10 @@ AC_DEFUN([FP_SETUP_PROJECT_VERSION], ProjectPatchLevel1=`echo $ProjectPatchLevel | sed 's/^\(@<:@^.@:>@*\)\(\.\{0,1\}\(.*\)\)$/\1/'` ProjectPatchLevel2=`echo $ProjectPatchLevel | sed 's/^\(@<:@^.@:>@*\)\(\.\{0,1\}\(.*\)\)$/\3/'` + # The project patchlevel1/2 is zero unless stated otherwise + test -z "$ProjectPatchLevel1" && ProjectPatchLevel1=0 + test -z "$ProjectPatchLevel2" && ProjectPatchLevel2=0 + AC_SUBST([ProjectPatchLevel1]) AC_SUBST([ProjectPatchLevel2]) diff --git a/testsuite/tests/driver/all.T b/testsuite/tests/driver/all.T index 907002fcf7..af81d61eb8 100644 --- a/testsuite/tests/driver/all.T +++ b/testsuite/tests/driver/all.T @@ -303,3 +303,4 @@ test('T20200loop', extra_files(['T20200loop']), multimod_compile, ['Datatypes', '-iT20200loop -O -v0']) test('T20316', normal, makefile_test, []) test('MultiRootsErr', normal, multimod_compile_fail, ['MultiRootsErr', 'MultiRootsErr']) +test('patch-level2', normal, compile, ['-Wcpp-undef']) diff --git a/testsuite/tests/driver/patch-level2.hs b/testsuite/tests/driver/patch-level2.hs new file mode 100644 index 0000000000..ae89fbed8e --- /dev/null +++ b/testsuite/tests/driver/patch-level2.hs @@ -0,0 +1,6 @@ +{-# LANGUAGE CPP #-} +module PatchLevel2 where + + +p2 = __GLASGOW_HASKELL_PATCHLEVEL2__ +p1 = __GLASGOW_HASKELL_PATCHLEVEL1__ |