[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to excl
From: |
Maxim Cournoyer |
Subject: |
[bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers. |
Date: |
Wed, 26 Jun 2024 15:28:11 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Hi Greg,
Greg Hogan <code@greghogan.com> writes:
> On Tue, Jun 25, 2024 at 11:15 AM Ludovic Courtès <ludo@gnu.org> wrote:
>>
>> Hi,
>>
>> Simon Tournier <zimon.toutoune@gmail.com> skribis:
>>
>> > +@lisp
>> > +(package
>> > + (name "python-scikit-learn")
>> > + ;; @dots{}
>> > + (properties '((lint-exclude-archival? . #t)
>> > + (lint-exclude-home-page? . #t))))
>>
>> To complement Maxim’s review, how about:
>>
>> (properties '((lint-excluded-checkers . (archival home-page))))
>>
>> ?
>>
>> Apart from that, the idea sounds reasonable to me.
>>
>> Thanks,
>> Ludo’.
>
> Could we not instead create a GUIX_LINT_OPTIONS, similar to
> GUIX_BUILD_OPTIONS? Then anyone wishing to universally exclude certain
> checkers (or disable network checks) on their own system would be free
> to do so.
That would be a good option to have too, on top of the other one.
> I find the current implementation confusing since I don't believe the
> project would accept a new or modified package missing the home page
> or with archiving disabled. Stated another way, to which Guix packages
> are we adding lint exclusions?
I don't think these exclusions should be committed in general to the
repo, except when we have for example the author of some software
explicitly requesting that SWH archival be disabled for it in Guix.
It may also be useful e.g. for some project that really don't have a
home page, to avoid a spurious lint warning in this case.
--
Thanks,
Maxim
- [bug#71697] [PATCH v2] guix: scripts: lint: Honor package property to exclude chercker., (continued)
[bug#71697] [PATCH v3 1/2] scripts: lint: Add 'dry-run' option., Simon Tournier, 2024/06/22
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Simon Tournier, 2024/06/22
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Maxim Cournoyer, 2024/06/23
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Ludovic Courtès, 2024/06/25
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Greg Hogan, 2024/06/25
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Ricardo Wurmus, 2024/06/26
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers.,
Maxim Cournoyer <=
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Greg Hogan, 2024/06/27
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Maxim Cournoyer, 2024/06/28
- [bug#71697] [PATCH v3 2/2] scripts: lint: Honor package property to exclude checkers., Dale Mellor, 2024/06/30
[bug#71697] [PATCH v3 1/2] scripts: lint: Add 'dry-run' option., Maxim Cournoyer, 2024/06/23