<div dir="ltr">Hi <span style="font-size:12.8000001907349px">Gábor,</span><div><br></div><div>instead of changing `<a href="http://ghc.mk">ghc.mk</a>`, please change the builder script to also run `<span style="font-size:12.8000001907349px">git clone git://</span><a href="http://git.haskell.org/ghc-tarballs.git" rel="noreferrer" target="_blank" style="font-size:12.8000001907349px">git.haskell.org/ghc-tarballs.git</a><span style="font-size:12.8000001907349px">`.</span></div><div><br></div><div><div><span style="font-size:12.8000001907349px">(Edit: while typing this, I saw you just did so)</span></div></div><div><br></div><div><span style="font-size:12.8000001907349px">The benefits are:</span></div><div><span style="font-size:12.8000001907349px">* the windows specific part of creating a source distribution also gets tested</span></div><div><span style="font-size:12.8000001907349px">* the release manager can run </span><span style="font-size:12.8000001907349px">`make sdist` on any platform, and they'll get a complete source distribution they can upload</span></div><div><div><br></div></div><div>Thomas</div><div><br></div><div class="gmail_extra"><div class="gmail_quote">On Sun, Jul 26, 2015 at 8:36 PM, Páli Gábor János <span dir="ltr"><<a href="mailto:pali.gabor@gmail.com" target="_blank">pali.gabor@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Thomas,<br>
<br>
2015-07-21 12:45 GMT+02:00 Thomas Miedema <<a href="mailto:thomasmiedema@gmail.com">thomasmiedema@gmail.com</a>>:<br>
> To get the sources:<br>
><br>
>    git clone --recursive git://<a href="http://git.haskell.org/ghc.git" rel="noreferrer" target="_blank">git.haskell.org/ghc.git</a><br>
>    cd ghc<br>
>    git clone git://<a href="http://git.haskell.org/ghc-tarballs.git" rel="noreferrer" target="_blank">git.haskell.org/ghc-tarballs.git</a> # Windows only<br>
<br>
Apparently, that last line is also needed even on non-Windows systems<br>
when one wants to make the "sdist" target, at least as I have observed<br>
on my builders, e.g. [1].<br>
<br>
So I have put together a small patch that aims to fix this -- mind<br>
you, I have not tested it on Windows, but theoretically it shall just<br>
work:<br>
<br>
--- a/<a href="http://ghc.mk" rel="noreferrer" target="_blank">ghc.mk</a><br>
+++ b/<a href="http://ghc.mk" rel="noreferrer" target="_blank">ghc.mk</a><br>
@@ -1183,9 +1183,14 @@ sdist-windows-tarballs: sdist-windows-tarballs-prep<br>
 sdist-testsuite: sdist-testsuite-prep<br>
        cd $(SRC_DIST_TESTSUITE_ROOT)        && "$(TAR_CMD)" chf -<br>
$(SRC_DIST_BASE_NAME) 2> testsuite_log             | $(TAR_COMP_CMD)<br>
-c > $(TOP)/$(SRC_DIST_TESTSUITE_TARBALL)<br>
<br>
+SDIST_TARGETS= sdist-ghc sdist-testsuite<br>
+<br>
+ifeq "$(Windows_Host)" "YES"<br>
+SDIST_TARGETS+= sdist-windows-tarballs<br>
+endif<br>
<br>
 .PHONY: sdist<br>
-sdist : sdist-ghc sdist-windows-tarballs sdist-testsuite<br>
+sdist : $(SDIST_TARGETS)<br>
<br>
 sdist-manifest : $(SRC_DIST_GHC_TARBALL)<br>
        tar tjf $(SRC_DIST_GHC_TARBALL) | sed<br>
"s|^ghc-$(ProjectVersion)/||" | sort >sdist-manifest<br>
diff --git a/utils/haddock b/utils/haddock<br>
<br>
Are there any objections to commit this?<br>
<br>
Thanks,<br>
Gábor<br>
<br>
[1] <a href="http://haskell.inf.elte.hu/builders/freebsd-amd64-head/700/15.html" rel="noreferrer" target="_blank">http://haskell.inf.elte.hu/builders/freebsd-amd64-head/700/15.html</a><br>
</blockquote></div><br></div></div>