Hi,
On Sat, Jan 27, 2024 at 10:49:42PM -0800, Greg Steuck wrote:
> Now that we landed ghc-9.6 is a good time to have `make test` fixed. It
> took some doing to have them all fixed upstream and even then some of the
> fixes are still not in 9.6. Anyway, OK?
Ok.
Ciao,
Kili
> ---
> lang/ghc/Makefile | 20 ++------------------
> lang/ghc/distinfo | 2 --
> 2 files changed, 2 insertions(+), 20 deletions(-)
>
> diff --git a/lang/ghc/Makefile b/lang/ghc/Makefile
> index 9aa4f19d65c..1612e816c88 100644
> --- a/lang/ghc/Makefile
> +++ b/lang/ghc/Makefile
> @@ -39,8 +39,8 @@ LIB_DEPENDS = converters/libiconv \
> devel/libffi
>
> BUILD_DEPENDS = archivers/bzip2 \
> - archivers/gtar \
> - textproc/py-sphinx${MODPY_FLAVOR}>=4.0.2
> + archivers/gtar
> +
> RUN_DEPENDS =
>
> SITES = https://downloads.haskell.org/ghc/${GHC_VERSION}/
> @@ -61,17 +61,7 @@ DISTFILES.boot = ${BINDISTFILE-${MACHINE_ARCH}}
> DISTFILES.noextract = hadrian-sources-${BIN_VER}.tar.gz
> SITES.noextract = https://openbsd.dead-parrot.de/distfiles/
>
> -# sphinx-rtd-theme update patch set
> -# mirrored from https://gitlab.haskell.org/ghc/ghc/-/commit/70526f5bd8886126f49833ef20604a2c6477780a.patch
> -# "GIT binary patch" (gpatch doesn't handle it, patch segfaults)
> -# (sorry! -sthen)
> -SITES.p = https://spacehopper.org/mirrors/
> -DISTFILES.p += ghc-rtd-update-70526f5bd8.diff
> EXTRACT_ONLY = ${DISTFILES} ${DISTFILES.boot}
> -BUILD_DEPENDS += devel/git
> -pre-patch:
> - cd ${WRKSRC}; env -i git apply ${FULLDISTDIR}/ghc-rtd-update-70526f5bd8.diff
> -
>
> # Substvars for all libraries, assuming that ghc-boot, ghc-boot-th,
> # ghc-heap and ghci will always have the same version number as ghc.
> @@ -136,9 +126,6 @@ CONFIGURE_ENV += CONF_GCC_LINKER_OPTS_STAGE0="${GHC_CC_OPTS}" \
> CONF_GCC_LINKER_OPTS_STAGE2="${GHC_CC_OPTS}" \
> CONF_CC_OPTS_STAGE2="${GHC_CC_OPTS}"
>
> -# Do not pick up gpatch
> -CONFIGURE_ENV += ac_cv_path_PatchCmd=/usr/bin/patch
> -
> MAKE_FLAGS += StripLibraries=YES \
> INSTALL_BIN_OPTS=-s \
> HSCOLOUR_SRCS=NO \
> @@ -147,9 +134,6 @@ MAKE_FLAGS += StripLibraries=YES \
> # haddock: ... <stdout>: commitBuffer: invalid argument (invalid character)
> MAKE_ENV = LC_ALL=en_US.UTF-8
>
> -# For mktexpk (via dvips when building the PostScript documentation):
> -PORTHOME = ${WRKDIR}
> -
> TEST_DEPENDS = print/ghostscript/gnu ${MODPY_RUN_DEPENDS}
>
> post-extract:
> diff --git a/lang/ghc/distinfo b/lang/ghc/distinfo
> index 2a2065aa251..3c92fd3f3be 100644
> --- a/lang/ghc/distinfo
> +++ b/lang/ghc/distinfo
> @@ -2,11 +2,9 @@ SHA256 (ghc/ghc-9.6.4.20240111-amd64.tar.xz) = CedJ29vBFZyl1e+DgcUqPfjHMDRKmEOsX
> SHA256 (ghc/ghc-9.6.4.20240111-shlibs-amd64.tar.gz) = Nb3trqnIF8H5kfKEkeGLr+sl4rPeFsbW/gfkelRprrY=
> SHA256 (ghc/ghc-9.6.4-src.tar.xz) = EL8luLBxdP3ZhotcDFbBfA7x7ctiR7S4ZL6TNlG/1MA=
> SHA256 (ghc/ghc-9.6.4-testsuite.tar.xz) = bhMoL76//b+gpJiJQ3REyakM/ldgxHlpzUJFhUwzjXM=
> -SHA256 (ghc/ghc-rtd-update-70526f5bd8.diff) = maCALPRglB5J9LJEQxBqMGdVbH4qK2gqVuaU5xmYdNU=
> SHA256 (ghc/hadrian-sources-9.6.4.20240111.tar.gz) = wMMJfyP7Pr6xjb/tj9Kz5iZugGr6+duMwJ23aGsUWy0=
> SIZE (ghc/ghc-9.6.4-src.tar.xz) = 29451856
> SIZE (ghc/ghc-9.6.4-testsuite.tar.xz) = 7075820
> SIZE (ghc/ghc-9.6.4.20240111-amd64.tar.xz) = 74706384
> SIZE (ghc/ghc-9.6.4.20240111-shlibs-amd64.tar.gz) = 3544885
> -SIZE (ghc/ghc-rtd-update-70526f5bd8.diff) = 4245853
> SIZE (ghc/hadrian-sources-9.6.4.20240111.tar.gz) = 2125322
> \ No newline at end of file
> --
> 2.43.0
>
> >From 11e6bf454004e8e60894c7259f8c3a8fbd49525d Mon Sep 17 00:00:00 2001
> From: Greg Steuck <greg@nest.cx>
> Date: Sat, 27 Jan 2024 19:16:24 -0800
> Subject: [PATCH 2/2] Fix `make test` in lang/ghc
>
> ---
> lang/ghc/Makefile | 31 ++++++++++---------
> .../patches/patch-compiler_GHC_Unit_State_hs | 2 +-
> .../patch-libraries_process_tests_all_T | 12 +++++++
> .../patches/patch-testsuite_driver_testlib_py | 10 +++---
> 4 files changed, 33 insertions(+), 22 deletions(-)
> create mode 100644 lang/ghc/patches/patch-libraries_process_tests_all_T
>
> diff --git a/lang/ghc/Makefile b/lang/ghc/Makefile
> index 1612e816c88..dc02ebab65a 100644
> --- a/lang/ghc/Makefile
> +++ b/lang/ghc/Makefile
> @@ -132,7 +132,7 @@ MAKE_FLAGS += StripLibraries=YES \
> BUILD_SPHINX_PDF=NO
>
> # haddock: ... <stdout>: commitBuffer: invalid argument (invalid character)
> -MAKE_ENV = LC_ALL=en_US.UTF-8
> +MAKE_ENV += LC_ALL=en_US.UTF-8
>
> TEST_DEPENDS = print/ghostscript/gnu ${MODPY_RUN_DEPENDS}
>
> @@ -162,17 +162,20 @@ post-patch:
> cd ${WRKSRC} && \
> ${MODPY_BIN} hadrian/bootstrap/bootstrap.py --no-archive -w ../bin/ghc \
> -s ${FULLDISTDIR}/hadrian-sources-${BIN_VER}.tar.gz
> -do-build:
> - cd ${WRKSRC} && \
> +
> +HADRIAN = \
> env -i ${AUTOCONF_ENV} ${MAKE_ENV} \
> _build/bin/hadrian '*.*.ghc.link.opts+=-optl-Wl,--no-execute-only' \
> - '*.*.ghc.link.opts+=-optl-Qunused-arguments' \
> - '*.*.ghc.link.opts+=-L/usr/local/lib' \
> - --docs=none \
> - --flavour=release${TRANSFORMER} \
> - --no-color \
> - --jobs=${MAKE_JOBS} \
> - binary-dist-dir
> + '*.*.ghc.link.opts+=-optl-Qunused-arguments' \
> + '*.*.ghc.link.opts+=-L/usr/local/lib' \
> + --docs=none \
> + --flavour=release${TRANSFORMER} \
> + --no-color \
> + --jobs=${MAKE_JOBS}
> +
> +do-build:
> + cd ${WRKSRC} && \
> + ${HADRIAN} binary-dist-dir
> ln -sf ${WRKSRC}/_build/bindist/ghc-${GHC_VERSION}{-x86_64-unknown-openbsd,}
>
> do-install:
> @@ -181,11 +184,9 @@ do-install:
> env -i ${MAKE_ENV} ${FAKE_SETUP} ${MAKE_PROGRAM} ${ALL_FAKE_FLAGS} -f ${MAKE_FILE} ${FAKE_TARGET}
>
> do-test:
> - ulimit -c 0 -n 1024 && \
> - cd ${WRKSRC}/testsuite/tests && \
> - exec ${SETENV} ${MAKE_ENV} LC_CTYPE=en_US.UTF-8 \
> - ${MAKE_PROGRAM} ${MAKE_FLAGS} \
> - PYTHON="${MODPY_BIN}"
> + ln -sf /usr/bin/false ${WRKDIR}/bin/git
> + cd ${WRKSRC} && \
> + ulimit -s 8192 -d 10485760 -n 1024 && env ${HADRIAN} test
>
> # It doesn't matter whether this is the actual date of the bootstrapper
> # build. It's just used to get different distfiles whenever new
> diff --git a/lang/ghc/patches/patch-compiler_GHC_Unit_State_hs b/lang/ghc/patches/patch-compiler_GHC_Unit_State_hs
> index 937671f8614..e75eb5e0e6c 100644
> --- a/lang/ghc/patches/patch-compiler_GHC_Unit_State_hs
> +++ b/lang/ghc/patches/patch-compiler_GHC_Unit_State_hs
> @@ -5,7 +5,7 @@ Uses $topdir/include as the very first place to find DerivedConstants.h
> Index: compiler/GHC/Unit/State.hs
> --- compiler/GHC/Unit/State.hs.orig
> +++ compiler/GHC/Unit/State.hs
> -@@ -661,7 +661,13 @@ initUnits logger dflags cached_dbs home_units = do
> +@@ -672,7 +672,13 @@ initUnits logger dflags cached_dbs home_units = do
> -- really need to use the platform constants but they have not been loaded.
> case lookupUnitId unit_state rtsUnitId of
> Nothing -> return Nothing
> diff --git a/lang/ghc/patches/patch-libraries_process_tests_all_T b/lang/ghc/patches/patch-libraries_process_tests_all_T
> new file mode 100644
> index 00000000000..eec5ef90114
> --- /dev/null
> +++ b/lang/ghc/patches/patch-libraries_process_tests_all_T
> @@ -0,0 +1,12 @@
> +Disable a test fixed in the next version:
> +https://github.com/haskell/process/issues/266
> +
> +Index: libraries/process/tests/all.T
> +--- libraries/process/tests/all.T.orig
> ++++ libraries/process/tests/all.T
> +@@ -50,4 +50,4 @@ test('process011',
> + compile_and_run, [''])
> +
> + test('T8343', js_broken(22349), compile_and_run, [''])
> +-test('processT251', js_broken(22349), compile_and_run, [''])
> ++test('processT251', [when(opsys('openbsd'), skip), js_broken(22349)], compile_and_run, [''])
> diff --git a/lang/ghc/patches/patch-testsuite_driver_testlib_py b/lang/ghc/patches/patch-testsuite_driver_testlib_py
> index 229cddc1799..a3243e8f046 100644
> --- a/lang/ghc/patches/patch-testsuite_driver_testlib_py
> +++ b/lang/ghc/patches/patch-testsuite_driver_testlib_py
> @@ -1,21 +1,19 @@
> Index: testsuite/driver/testlib.py
> --- testsuite/driver/testlib.py.orig
> +++ testsuite/driver/testlib.py
> -@@ -2146,6 +2146,13 @@ def compare_outputs(way: WayName,
> +@@ -2146,6 +2146,11 @@ def compare_outputs(way: WayName,
> actual_raw = read_no_crs(actual_path)
> actual_str = normaliser(actual_raw)
>
> + # Strip OpenBSDs linker warnings about strcpy(3) & friends.
> + if config.os == 'openbsd':
> -+ actual_str = re.sub('.*: In function .*:\n+(.*:[0-9]+:[0-9]+:\n)?:? *warning: .* is .* misused, please use .*\n',
> -+ '', actual_str)
> -+ actual_str = re.sub('.*/libgmp\.so\.[0-9]*\.[0-9]*: warning: .* is .* misused, please use .*\n',
> ++ actual_str = re.sub('.*warning:.* misused, please use .*\n',
> + '', actual_str)
> +
> # See Note [Output comparison].
> if whitespace_normaliser(expected_str) == whitespace_normaliser(actual_str):
> return True
> -@@ -2162,7 +2169,7 @@ def compare_outputs(way: WayName,
> +@@ -2162,7 +2167,7 @@ def compare_outputs(way: WayName,
>
> if config.verbose >= 1 and _expect_pass(way):
> # See Note [Output comparison].
> @@ -24,7 +22,7 @@ Index: testsuite/driver/testlib.py
> actual_normalised_path),
> stdout=diff_file,
> print_output=True)
> -@@ -2170,7 +2177,7 @@ def compare_outputs(way: WayName,
> +@@ -2170,7 +2175,7 @@ def compare_outputs(way: WayName,
> # If for some reason there were no non-whitespace differences,
> # then do a full diff
> if r == 0:
> --
> 2.43.0
No comments:
Post a Comment