guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#29017] [PATCH] gnu: Add gwl.


From: Ludovic Courtès
Subject: [bug#29017] [PATCH] gnu: Add gwl.
Date: Thu, 26 Oct 2017 15:41:42 -0700
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

Hello!

Roel Janssen <address@hidden> skribis:

> I would like to add GWL.  For a lack of a better place, I chose the
> bioinformatics module because that's where we use this package.

Awesome!  \o/

Perhaps that could go to package-management.scm?

> From eaa22755769973ba30600ae96bb92a5d6bad11dd Mon Sep 17 00:00:00 2001
> From: Roel Janssen <address@hidden>
> Date: Thu, 26 Oct 2017 22:46:50 +0200
> Subject: [PATCH] gnu: Add gwl.
>
> * gnu/packages/bioinformatics.scm (gwl): New variable.

[...]

> +(define-public gwl
> +  (package
> +    (name "gwl")
> +    (version "0.1.0")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "https://git.roelj.com/attachments/";
> +                                  "d804e310-b9c0-44b1-bd18-c7505e9f9d9b"))

Do you have a more “standard” URL, such as a cgit snapshot?  If not
that’s fine.

> +    (inputs
> +     `(("guix" ,guix)
> +       ("guile" ,guile-2.2)
> +       ("guile-commonmark" ,guile-commonmark)))

It might be that you either need to propagate Guix and CommonMark, or to
wrap the binaries that GWL provides (as is done in the ‘guix’ package),
no?

Otherwise LGTM, thank you!

Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]