[debbugs-tracker] bug#31736: closed ([PATCH] Add an opam importer)

[GNU bug Tracking System]

Your message dated Tue, 10 Jul 2018 13:47:58 +0200
with message-id <address@hidden>
and subject line Re: [bug#31736] [PATCH] Add an opam importer
has caused the debbugs.gnu.org bug report #31736,
regarding [PATCH] Add an opam importer
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
31736: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=31736
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: [PATCH] Add an opam importer Date: Wed, 6 Jun 2018 19:23:29 +0200

Hi, this patch adds an importer for ocaml packages from the opam
repository.

0001-guix-Add-opam-importer.patch
Description: Text Data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: [bug#31736] [PATCH] Add an opam importer
>
>
>
>
> Date: 
>
> Tue, 10 Jul 2018 13:47:58 +0200
>
>
>
>
> Le Mon, 09 Jul 2018 14:44:20 +0200,
> address@hidden (Ludovic Courtès) a écrit :
>
> > Hello Julien,
> > 
> > Julien Lepiller <address@hidden> skribis:
> > 
> > > Here is a new version. What do you think?  
> > 
> > I think it looks great.  :-)
> > 
> > > From b3fd75a804a4264cb083584591a3e589886d96c8 Mon Sep 17 00:00:00
> > > 2001 From: Julien Lepiller <address@hidden>
> > > Date: Wed, 6 Jun 2018 19:14:39 +0200
> > > Subject: [PATCH] guix: Add opam importer.
> > >
> > > * guix/scripts/import.scm (importers): Add opam.
> > > * guix/scripts/import/opam.scm: New file.
> > > * guix/import/opam.scm: New file.
> > > * tests/opam.scm: New file.
> > > * Makefile.am: Add them.
> > > * doc/guix.texi (Invoking guix import): Document it.  
> > 
> > [...]
> > 
> > > address@hidden opam
> > > address@hidden opam
> > > address@hidden ocaml
> > > +Import metadata from the @uref{https://opam.ocaml.org/, Opam}
> > > package +repository used by the OCaml community.  
> > 
> > Nitpick: the proper spelling is “OPAM” and “OCaml”, so I think you
> > need to fix these above (in the concept index as well.)
> > 
> > Otherwise LGTM!
> > 
> > As a next step, you can add an OPAM updater, which will hopefully take
> > you no more than a few lines of code (see the bottom of cpam.scm, for
> > example.)
> > 
> > :-)
> > 
> > Thank you!
> > 
> > Ludo’.
>
> Pushed as b24443bff9f9f3f36353eea2ef35e6dc3745a417
>
>
> --- End Message ---