* bug#18868: tests/guix-lint.sh fails
@ 2014-10-28 9:32 Alex Kost
2014-10-28 14:33 ` Eric Bavier
0 siblings, 1 reply; 4+ messages in thread
From: Alex Kost @ 2014-10-28 9:32 UTC (permalink / raw)
To: 18868
“make check TESTS=tests/guix-lint.sh” is now failing. AFAIU it happens
because now (after commit 3c42965) descriptions may start with a
lower-case letter, so some checks from “tests/guix-lint.sh” need to be
adjusted.
Sorry, I don't have a patch, just a report.
--
Alex
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#18868: tests/guix-lint.sh fails
2014-10-28 9:32 bug#18868: tests/guix-lint.sh fails Alex Kost
@ 2014-10-28 14:33 ` Eric Bavier
2014-10-28 16:30 ` Alex Kost
0 siblings, 1 reply; 4+ messages in thread
From: Eric Bavier @ 2014-10-28 14:33 UTC (permalink / raw)
To: Alex Kost; +Cc: 18868
Alex Kost writes:
> “make check TESTS=tests/guix-lint.sh” is now failing. AFAIU it happens
> because now (after commit 3c42965) descriptions may start with a
> lower-case letter, so some checks from “tests/guix-lint.sh” need to be
> adjusted.
>
> Sorry, I don't have a patch, just a report.
Thanks for the report. I failed to run that test after the last
adjustment I made to the linters. Commit 791e012 adjusts the test case
to the latest linter rules. Could you confirm?
--
Eric Bavier
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#18868: tests/guix-lint.sh fails
2014-10-28 14:33 ` Eric Bavier
@ 2014-10-28 16:30 ` Alex Kost
2014-11-02 20:35 ` Ludovic Courtès
0 siblings, 1 reply; 4+ messages in thread
From: Alex Kost @ 2014-10-28 16:30 UTC (permalink / raw)
To: Eric Bavier; +Cc: 18868
Eric Bavier (2014-10-28 17:33 +0300) wrote:
> Alex Kost writes:
>
>> “make check TESTS=tests/guix-lint.sh” is now failing. AFAIU it happens
>> because now (after commit 3c42965) descriptions may start with a
>> lower-case letter, so some checks from “tests/guix-lint.sh” need to be
>> adjusted.
>>
>> Sorry, I don't have a patch, just a report.
>
> Thanks for the report. I failed to run that test after the last
> adjustment I made to the linters. Commit 791e012 adjusts the test case
> to the latest linter rules. Could you confirm?
Absolutely, it works now. Thanks for fixing it.
--
Alex
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#18868: tests/guix-lint.sh fails
2014-10-28 16:30 ` Alex Kost
@ 2014-11-02 20:35 ` Ludovic Courtès
0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2014-11-02 20:35 UTC (permalink / raw)
To: Alex Kost; +Cc: Eric Bavier, 18868-close
Closing the bug. Thanks to both of you.
Ludo’.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-11-02 20:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-28 9:32 bug#18868: tests/guix-lint.sh fails Alex Kost
2014-10-28 14:33 ` Eric Bavier
2014-10-28 16:30 ` Alex Kost
2014-11-02 20:35 ` Ludovic Courtès
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).