guix lint doesn't actually fail on some errors

OpenSubmitted by Danny Milosavljevic.
Details
2 participants
  • Danny Milosavljevic
  • Ludovic Courtès
Owner
unassigned
Severity
normal
D
D
Danny Milosavljevic wrote on 27 Jul 2016 00:25
(address . bug-guix@gnu.org)
20160727002502.4acf8f47@scratchpost.org
guix lint doesn't actually fail on some errors.
$ guix lint perl-io-socket-inet6 ; echo $?;;; note: source file /x/home/dannym/src/guix/gnu/packages/networking.scm;;; newer than compiled /x/home/dannym/src/guix/gnu/packages/networking.go;;; note: source file /x/home/dannym/src/guix/gnu/packages/mail.scm;;; newer than compiled /x/home/dannym/src/guix/gnu/packages/mail.go;;; note: source file /x/home/dannym/src/guix/gnu/packages/ldc.scm;;; newer than compiled /x/home/dannym/src/guix/gnu/packages/ldc.gognu/packages/networking.scm:437:1: perl-io-socket-inet6-2.72: line 464 is way too long (96 characters)gnu/packages/networking.scm:437:1: perl-io-socket-inet6-2.72: line 483 is way too long (92 characters)0^--- ooops
L
L
Ludovic Courtès wrote on 27 Jul 2016 22:57
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 24083@debbugs.gnu.org)
87mvl2x0ic.fsf@gnu.org
Danny Milosavljevic <dannym@scratchpost.org> skribis:
Toggle quote (2 lines)> guix lint doesn't actually fail on some errors.
It never pretended to exit with non-zero when warnings are emitted. :-)(These are warnings, not errors.)
Ludo’.
D
D
Danny Milosavljevic wrote on 27 Jul 2016 23:12
(name . Ludovic Courtès)(address . ludo@gnu.org)(address . 24083@debbugs.gnu.org)
20160727231222.66849389@scratchpost.org
On Wed, 27 Jul 2016 22:57:15 +0200ludo@gnu.org (Ludovic Courtès) wrote:
Toggle quote (7 lines)> Danny Milosavljevic <dannym@scratchpost.org> skribis:> > > guix lint doesn't actually fail on some errors. > > It never pretended to exit with non-zero when warnings are emitted. :-)> (These are warnings, not errors.)
Too bad :)
I wrote a little helper script which was supposed to prevent me from posting packages that don't build or lint. Then I found out that the lint actually doesn't fail on warnings. So now my script is only half as useful (the builder does fail on build error).
It would be nice to have a way to also check the lint status.
?