Deleting sync-all

Tamar Christina lonetiger at gmail.com
Tue Jul 28 03:48:35 UTC 2015


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 <thomasmiedema at gmail.com>
Sent: ‎7/‎27/‎2015 18:25
To: Tamar Christina <lonetiger at gmail.com>
Cc: Páli Gábor János <pali.gabor at gmail.com>; ghc-devs at haskell.org
Subject: Re: Deleting sync-all

Hi Tamar,

feel free to take over that patch <https://phabricator.haskell.org/D339>,
or extract the parts you like. It currently does (too) many things, which
is part of the reason why it stalled. See my comments in #9218
<https://ghc.haskell.org/trac/ghc/ticket/9218>. Do not wait to get rid of
the perl dependency first, because it is yet unclear how to proceed (#9832
<https://ghc.haskell.org/trac/ghc/ticket/9832>).

Thomas


On Mon, Jul 27, 2015 at 2:34 PM, Tamar Christina <lonetiger at gmail.com>
wrote:

> 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 <thomasmiedema at gmail.com>
> Sent: ‎7/‎27/‎2015 10:41
> To: Páli Gábor János <pali.gabor at gmail.com>
> Cc: ghc-devs at 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 <pali.gabor at gmail.com>
> wrote:
>
>> Hi Thomas,
>>
>> 2015-07-21 12:45 GMT+02:00 Thomas Miedema <thomasmiedema at gmail.com>:
>> > 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
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/ghc-devs/attachments/20150727/986026c7/attachment-0001.html>


More information about the ghc-devs mailing list