all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Philip Munksgaard" <philip@munksgaard.me>
To: "John Kehayias" <john.kehayias@protonmail.com>,
	"49418-done@debbugs.gnu.org" <49418-done@debbugs.gnu.org>
Subject: bug#49418: Importing haskell packages from hackage doesn't apply metadata revisions
Date: Wed, 07 Jul 2021 11:52:05 +0200	[thread overview]
Message-ID: <affbe2ca-9725-448a-8e2d-943dfd851196@www.fastmail.com> (raw)
In-Reply-To: <1d5f0af1-ed89-4b03-b845-207fd8731a62@www.fastmail.com>

On Wed, 7 Jul 2021, at 07:58, Philip Munksgaard wrote:
> On Wed, 7 Jul 2021, at 05:26, John Kehayias via Bug reports for GNU Guix wrote:
> > Actually, this does exist in the Haskell build system in Guix, but 
> > seems to be undocumented and not used by the importer. You can add the 
> > following to the arguments (in the bootstrap package in this case) to 
> > use a metadata revision:
> > 
> > #:cabal-revision ("2" "0a6j3313vz7n7dn8abddyib4jggblaq89f87ib4imdwjxjajbm33")
> > 
> > The hash is from running guix hash file (where file = 2.cabal in this 
> > case, downloaded from Hackage). This should be part of the importer, to 
> > specify a revision or by default grab the latest, I would say.
> > 
> > (and I'm guessing you know this is packaged in guix as 
> > integer-logarithms, without the "ghc-" prefix for some reason; not the 
> > only package like that I've noticed)
> > 
> 
> Ah yes, good catch! I agree that the fix should be to amend the 
> importer, such that it finds out about these revisions and 
> automatically uses the latest one.

Actually, upon closer inspection, that's exactly what it does! Instead of actually using the importer in my original example (as I claimed), I had actually just modified the code from gnu/packages/haskell-xyz.scm. Doing a fresh import correctly picks up that there is a new revision of the cabal file and produces the right derivation. I'll close this issue.




  reply	other threads:[~2021-07-07  9:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-05 14:09 bug#49418: Importing haskell packages from hackage doesn't apply metadata revisions Philip Munksgaard
2021-07-05 15:32 ` John Kehayias via Bug reports for GNU Guix
2021-07-07  3:26   ` John Kehayias via Bug reports for GNU Guix
2021-07-07  5:58     ` Philip Munksgaard
2021-07-07  9:52       ` Philip Munksgaard [this message]
2021-07-07 14:37         ` John Kehayias via Bug reports for GNU Guix

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=affbe2ca-9725-448a-8e2d-943dfd851196@www.fastmail.com \
    --to=philip@munksgaard.me \
    --cc=49418-done@debbugs.gnu.org \
    --cc=john.kehayias@protonmail.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.