[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/4] lint: Add 'inputs-should-be-sorted' check.
From: |
Eric Bavier |
Subject: |
Re: [PATCH 3/4] lint: Add 'inputs-should-be-sorted' check. |
Date: |
Thu, 21 Jul 2016 08:17:10 -0500 |
On Tue, 19 Jul 2016 18:49:15 +0200
David Craven <address@hidden> wrote:
> * guix/scripts/lint.scm (check-inputs-should-be-sorted): Add it.
> (lint-checker): Add it.
> ---
> guix/scripts/lint.scm | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/guix/scripts/lint.scm b/guix/scripts/lint.scm
> index d64a066..d3a82c9 100644
> --- a/guix/scripts/lint.scm
> +++ b/guix/scripts/lint.scm
> @@ -220,6 +220,21 @@ by two spaces; possible infraction~p at ~{~a~^, ~}")
> (when (string-suffix? ":bin" package-name) (warn
> package-name)))
> labels)))))))
>
> +(define (check-inputs-should-be-sorted package)
> + ;; Emit a warning if inputs, native-inputs or propagated-inputs are not
> sorted.
I don't think this needs to be a thing. While I personally added the
code in the cpan importer to sort inputs by name, I recognize that
there are also valid reasons to not do that in some packages. E.g.
grouping a set of inputs that are all related to or required for some
aspect of a package's functionality, etc.
`~Eric
pgpaMMqYyQLQp.pgp
Description: OpenPGP digital signature
- [PATCH 0/4] Dipping my toes into guix source., David Craven, 2016/07/19
- [PATCH 3/4] lint: Add 'inputs-should-be-sorted' check., David Craven, 2016/07/19
- Re: [PATCH 3/4] lint: Add 'inputs-should-be-sorted' check.,
Eric Bavier <=
- [PATCH 4/4] environment: Add '--debug' flag., David Craven, 2016/07/19
- Re: [PATCH 4/4] environment: Add '--debug' flag., Thompson, David, 2016/07/19
- Re: [PATCH 4/4] environment: Add '--debug' flag., Tobias Geerinckx-Rice, 2016/07/19
- Re: [PATCH 4/4] environment: Add '--debug' flag., David Craven, 2016/07/19
- Re: [PATCH 4/4] environment: Add '--debug' flag., David Craven, 2016/07/19
- Re: [PATCH 4/4] environment: Add '--debug' flag., Thompson, David, 2016/07/19
- Re: [PATCH 4/4] environment: Add '--debug' flag., David Craven, 2016/07/19
- Re: [PATCH 4/4] environment: Add '--debug' flag., Leo Famulari, 2016/07/20
- Re: [PATCH 4/4] environment: Add '--debug' flag., Ludovic Courtès, 2016/07/20