[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/7] emacs: Add 'guix-browse-license-url' command.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 3/7] emacs: Add 'guix-browse-license-url' command. |
Date: |
Sat, 23 Jan 2016 18:13:18 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Alex Kost <address@hidden> skribis:
> * emacs/guix-main.scm (license-names, license-name->license)
> (license-name->uri): New procedures.
> * emacs/guix-read.el (guix-license-names, guix-read-license-name): New
> procedures.
> * emacs/guix-license.el: New file.
> * emacs.am (ELFILES): Add it.
[...]
> +;;; Licenses
I’d do:
(define %licenses
(delay (module-map thing from previous message…)))
> +(define (license-names)
> + "Return a list of names of available licenses."
> + (map license-name %licenses))
So here it’d become: (force %licenses).
Other than that, LGTM!
Ludo’.
- Re: [PATCH 4/7] emacs: Buttonize licenses in "Package Info"., (continued)
- [PATCH 5/7] emacs: Add 'guix-packages-by-license' command., Alex Kost, 2016/01/22
- [PATCH 1/7] emacs: Rename 'file-path' to 'file-name'., Alex Kost, 2016/01/22
- [PATCH 6/7] emacs: list: Support displaying URL links., Alex Kost, 2016/01/22
- [PATCH 7/7] emacs: Add interface for licenses., Alex Kost, 2016/01/22
- [PATCH 3/7] emacs: Add 'guix-browse-license-url' command., Alex Kost, 2016/01/22
- [PATCH 2/7] licenses: Add a list of licenses., Alex Kost, 2016/01/22