[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#12198: [PATCH] tests: fixup: make distcheck-override-infodir pass ag
From: |
Peter Rosin |
Subject: |
bug#12198: [PATCH] tests: fixup: make distcheck-override-infodir pass again |
Date: |
Tue, 14 Aug 2012 19:49:47 +0200 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20120713 Thunderbird/14.0 |
On 2012-08-14 18:53, Stefano Lattarini wrote:
> On 08/14/2012 06:44 PM, Stefano Lattarini wrote:
>> Hi Peter, thanks for the report and the fix.
>>
>> On 08/14/2012 03:25 PM, Peter Rosin wrote:
>>>
>>> Ok to push this fix to maint?
>>>
>>> Cheers,
>>> Peter
>>>
>>> [Subject] tests: fixup: make distcheck-override-infodir pass again
>>>
>> I'd suggest being a little more precise:
>>
>> tests: fixup: avoid spurious failure on Cygwin for
>> distcheck-override-infodir
>>
> Forget it, I've seen that the failure is cross-system (reported in automake
> bug#12198). Your origin summary line was correct, sorry. However, may I ask
> that, the next time you send the fix for a bug, you CC: the relevant bug
> report (as I'm doing now) and add a reference to the bug number in the commit
> message? In this case, change it to read:
>
> tests: fixup: make distcheck-override-infodir pass again
>
> Fixes Automake bug#12198.
>
> * t/distcheck-override-infodir.sh (main.texi): Remove all leading
> cruft added by commit v1.12.2-96-g133307b "maintcheck: fix spurious
> warnings".
>
Done, but note that I posted the patch before the bug report arrived in
my mailbox. I'm still causal, so it would have been a neat trick to
include that bug number from the start. :-)
>>> diff --git a/t/distcheck-override-infodir.sh
>>> b/t/distcheck-override-infodir.sh
>>> index 74f6c67..eb41a37 100755
>>> --- a/t/distcheck-override-infodir.sh
>>> +++ b/t/distcheck-override-infodir.sh
>>> @@ -39,7 +39,7 @@ installcheck-local:
>>> END
>>>
>>> # Protect with leading " # " to avoid spurious maintainer-check failures.
>>> -sed 's/^ #//' > main.texi << 'END'
>>> +sed 's/^ # //' > main.texi << 'END'
>>>
>> I'd say we might be even safer and do 's/^ *# *//'. OK?
>>
>>> # \input texinfo
>>> # @setfilename main.info
>>> # @settitle main
>>>
>> ACK with those two nits addressed.
>>
> Ditto :-)
Pushed with nits fixed...
Cheers,
Peter