[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete br
From: |
Marius Bakke |
Subject: |
Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages) |
Date: |
Thu, 08 Aug 2019 18:08:17 +0200 |
User-agent: |
Notmuch/0.29.1 (https://notmuchmail.org) Emacs/26.2 (x86_64-pc-linux-gnu) |
Timothy Sample <address@hidden> writes:
> Hi,
>
> Robert Vollmert <address@hidden> writes:
>
>> On 8. Aug 2019, at 15:12, Marius Bakke <address@hidden> wrote:
>>> I have one comment about the series: we've disabled tests on some
>>> packages that have been broken "forever" on i686. It would be better to
>>> do so selectively on just the affected architectures. I.e.:
>>>
>>> #:tests? (if (string-prefix? "i686" (%current-system))
>>> #f
>>> #t)
>>>
>>> Preferably with a comment about why they need to be disabled. That way,
>>> we will still notice when something breaks on other architectures. Can
>>> you try it Rob?
>>
>> I don’t intend to, because I think the effort is better spent elsewhere.
>> But do make the change if you like!
>>
>> The rough plan from my point of view would be to aim for an upgrade of
>> the haskell packages to build with ghc-8.6 from a recent stackage LTS
>> set, and reevaluate skipped tests across the set while doing that or
>> once that’s done.
>
> This was in the back of my mind, too. Stackage LTS 14.0 (built on top
> of GHC 8.6.5) was released three days ago. Upgrading will involve
> sweeping changes to the whole set of Haskell packages, giving us lots of
> opportunities to revisit these failing tests. If we still have problems
> with the i686 tests, we can make them conditional then.
>
> Thanks for pointing this out though, Marius. I had thought about making
> them conditional when reviewing, but I second guessed myself because we
> have a lot of packages with comments like “tests fail on architecture X”
> followed by an absolute “#:tests? #f”. If this comes up in the future,
> I’ll just go ahead and make the tests conditional.
OK. Thanks to both of you for clarifying :-)
signature.asc
Description: PGP signature
- Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Robert Vollmert, 2019/08/01
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Timothy Sample, 2019/08/06
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Robert Vollmert, 2019/08/06
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Robert Vollmert, 2019/08/06
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Marius Bakke, 2019/08/07
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Timothy Sample, 2019/08/07
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Marius Bakke, 2019/08/08
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Robert Vollmert, 2019/08/08
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages), Timothy Sample, 2019/08/08
- Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages),
Marius Bakke <=