[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] syntax-check: remove SPACE-TAB sequence
From: |
Eric Blake |
Subject: |
Re: [PATCH 3/3] syntax-check: remove SPACE-TAB sequence |
Date: |
Fri, 16 Nov 2012 08:45:39 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121029 Thunderbird/16.0.2 |
On 11/16/2012 04:48 AM, Stefano Lattarini wrote:
> Hi Peter.
>
> On 11/16/2012 12:38 PM, Peter Rosin wrote:
>> On 2012-11-16 11:37, Stefano Lattarini wrote:
>>> * lib/autoconf/c.m4 (_AC_PROG_CC_C99): Here. Was causing a
>>> failure in the 'space_tab' syntax check.
>>>
>>> Signed-off-by: Stefano Lattarini <address@hidden>
>>> ---
>>> lib/autoconf/c.m4 | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/lib/autoconf/c.m4 b/lib/autoconf/c.m4
>>> index 0802f85..b1edf17 100644
>>> --- a/lib/autoconf/c.m4
>>> +++ b/lib/autoconf/c.m4
>>> @@ -1349,7 +1349,7 @@ dnl Try
>>> dnl GCC -std=gnu99 (unused restrictive modes: -std=c99
>>> -std=iso9899:1999)
>>> dnl IBM XL C -qlanglvl=extc1x (V12.1; does not pass C11 test)
>>> dnl IBM XL C -qlanglvl=extc99
>>> -dnl (pre-V12.1; unused restrictive mode: -qlanglvl=stdc99)
>>> +dnl (pre-V12.1; unused restrictive mode: -qlanglvl=stdc99)
>>> dnl HP cc -AC99
>>> dnl Intel ICC -std=c99, -c99 (deprecated)
>>> dnl IRIX -c99
>>>
>>
>> But the surrounding uses TABs, not spaces.
>>
> Well spotted. I've fixed that locally.
Autoconf has not yet made the global switch away from TABs, but I'm
willing to go that route to match what other GNU projects have done. In
the meantime, fixing the syntax check makes sense, so ACK to either what
you proposed, or your fixed version with local TAB fixes in this area.
--
Eric Blake address@hidden +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
Re: [PATCH 1/3] build: require autoconf >= 2.62, and related simplifications, Eric Blake, 2012/11/16