unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
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

  reply	other threads:[~2019-08-06  7:04 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <F5E8BA95-BE79-4579-B881-AE0862B69D3A@vllmrt.net>
     [not found] ` <87muh2ckdn.fsf@ngyro.com>
2019-08-01 16:23   ` Please merge wip-haskell-updates (Re: [bug#36807] remove obsolete broken haskell packages) 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-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

  List information: https://guix.gnu.org/

* 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 public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).