From: Robert Vollmert <rob@vllmrt.net>
To: Timothy Sample <samplet@ngyro.com>
Cc: guix-devel@gnu.org, 36807@debbugs.gnu.org
Subject: Re: Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages)
Date: Tue, 6 Aug 2019 09:04:39 +0200 [thread overview]
Message-ID: <3C8EAC60-7073-4528-BC5D-548A018F08D8@vllmrt.net> (raw)
In-Reply-To: <87mugmq5ll.fsf@ngyro.com>
Hi Timothy,
> On 6. Aug 2019, at 06:29, Timothy Sample <samplet@ngyro.com> wrote:
>
>> #36663: adding elm compiler dependencies (just a few extra ghc
>> packages)
>
> These commits seem to be in the wrong order. I think I can untangle
> them, though.
>
>> #36692: GHC version 8.6.5 (just as a package for now, not used to build
>> anything)
>
> I made some bigger changes here. Mostly, I made use of
> “substitute-keyword-arguments” to reuse more code from “ghc-8.4”.
>
> Why do you use “patch” instead of “substitute*” to disable the failing
> tests? I see from your previous patches that you used to do it with
> “substitute*”.
It would be ok to go back to the old state. I moved to a patch over the
process of getting the build to pass, which involved skipping more tests.
That said, substitute has several downsides compared to patches:
- patch is easier to read
- patch doesn’t fail silently
>> no ticket: Skip tests for three Haskell packages that fail on i686 only
>> (and seem harmless): ghc-trifecta, ghc-yaml, ghc-libmpd-haskell.
>
> This seems reasonable to me, though I suppose it would be better to only
> skip them when building for i686. It looks like we only do this
> rarely (e.g., the “icu4c” package), so maybe it’s not a big deal.
I’ll keep that in mind for next time I run into a similar issue.
> Is there any more info about “ghc-trifecta”? The other two have nice
> comments that tell me that upstream is aware of the problem, and that it
> might be fixed in the future.
That one is a rather opaque build failure kind of thing related to doctests:
They fail to build on i686:
doctests:
ByteCodeLink.lookupCE
During interactive linking, GHCi couldn't find the following symbol:
lenszm4zi16zi1zmJLUwQ4zzqmnaKkc25AByaCJ_ControlziLensziTH_makeClassy_closure
This may be due to you not asking GHCi to load extra object files,
archives or DLLs needed by your current session. Restart GHCi, specifying
the missing library using the -L/path/to/object/dir and -lmissinglibname
flags, or simply by naming the relevant files on the GHCi command line.
Alternatively, this link failure might indicate a bug in GHCi.
If you suspect the latter, please send a bug report to:
glasgow-haskell-bugs@haskell.org
Test suite doctests: FAIL
I spent a bit of time digging, then gave up.
Thanks for the review.
Robert
next prev parent reply other threads:[~2019-08-06 7:04 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-25 11:40 [bug#36807] remove obsolete broken haskell packages Robert Vollmert
2019-07-25 11:45 ` [bug#36807] [PATCH] gnu: Remove ghc-regex-tdfa-rc (broken and unused) Robert Vollmert
2019-07-25 11:46 ` [bug#36807] [PATCH] gnu: Remove ghc-haddock-test " Robert Vollmert
2019-07-25 13:29 ` [bug#36807] remove obsolete broken haskell packages Timothy Sample
2019-08-01 16:23 ` Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages) Robert Vollmert
2019-08-01 16:23 ` [bug#36807] " Robert Vollmert
2019-08-06 4:29 ` Timothy Sample
2019-08-06 7:04 ` Robert Vollmert [this message]
2019-08-08 3:42 ` Timothy Sample
2019-08-08 4:28 ` bug#36807: " Timothy Sample
2019-08-06 7:49 ` Robert Vollmert
2019-08-07 11:19 ` Marius Bakke
2019-08-08 3:47 ` Timothy Sample
2019-08-08 13:12 ` Marius Bakke
2019-08-08 13:42 ` Robert Vollmert
2019-08-08 15:54 ` Timothy Sample
2019-08-08 16:08 ` Marius Bakke
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3C8EAC60-7073-4528-BC5D-548A018F08D8@vllmrt.net \
--to=rob@vllmrt.net \
--cc=36807@debbugs.gnu.org \
--cc=guix-devel@gnu.org \
--cc=samplet@ngyro.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.