
Hi Thomas,
Sure I can take care of that part. I'll make a separate issue for it. I
also think we should do the first part as well eventually (using mingw-w
for for both x64 and x86_64) as that solves #10435 .
Regards,
Tamar
------------------------------
From: Thomas Miedema
Hi
There is a sorta in limbo differential from gintas that would have solved this by doing it in the configure script.
I liked that solution. As far as I know the patch was delayed because the reviewers suggested we get rid of the perl dependency first.
Regards, Tamar ------------------------------ From: Thomas Miedema
Sent: 7/27/2015 10:41 To: Páli Gábor János Cc: ghc-devs@haskell.org Subject: Re: Deleting sync-all Hi Gábor,
instead of changing `ghc.mk`, please change the builder script to also run `git clone git://git.haskell.org/ghc-tarballs.git`.
(Edit: while typing this, I saw you just did so)
The benefits are: * the windows specific part of creating a source distribution also gets tested * the release manager can run `make sdist` on any platform, and they'll get a complete source distribution they can upload
Thomas
On Sun, Jul 26, 2015 at 8:36 PM, Páli Gábor János
wrote: Hi Thomas,
2015-07-21 12:45 GMT+02:00 Thomas Miedema
: To get the sources:
git clone --recursive git://git.haskell.org/ghc.git cd ghc git clone git://git.haskell.org/ghc-tarballs.git # Windows only
Apparently, that last line is also needed even on non-Windows systems when one wants to make the "sdist" target, at least as I have observed on my builders, e.g. [1].
So I have put together a small patch that aims to fix this -- mind you, I have not tested it on Windows, but theoretically it shall just work:
--- a/ghc.mk +++ b/ghc.mk @@ -1183,9 +1183,14 @@ sdist-windows-tarballs: sdist-windows-tarballs-prep sdist-testsuite: sdist-testsuite-prep cd $(SRC_DIST_TESTSUITE_ROOT) && "$(TAR_CMD)" chf - $(SRC_DIST_BASE_NAME) 2> testsuite_log | $(TAR_COMP_CMD) -c > $(TOP)/$(SRC_DIST_TESTSUITE_TARBALL)
+SDIST_TARGETS= sdist-ghc sdist-testsuite + +ifeq "$(Windows_Host)" "YES" +SDIST_TARGETS+= sdist-windows-tarballs +endif
.PHONY: sdist -sdist : sdist-ghc sdist-windows-tarballs sdist-testsuite +sdist : $(SDIST_TARGETS)
sdist-manifest : $(SRC_DIST_GHC_TARBALL) tar tjf $(SRC_DIST_GHC_TARBALL) | sed "s|^ghc-$(ProjectVersion)/||" | sort >sdist-manifest diff --git a/utils/haddock b/utils/haddock
Are there any objections to commit this?
Thanks, Gábor
[1] http://haskell.inf.elte.hu/builders/freebsd-amd64-head/700/15.html
participants (1)
-
Tamar Christina