[PATCH] Properly check --enable-compressed-debug-sections={yes,all}

[H.J. Lu]

On Thu, Oct 22, 2015 at 8:58 AM, Andreas Schwab <schwab@suse.de> wrote:
> "H.J. Lu" <hjl.tools@gmail.com> writes:
>
>> On Thu, Oct 22, 2015 at 8:41 AM, Andreas Schwab <schwab@suse.de> wrote:
>>> "H.J. Lu" <hjl.tools@gmail.com> writes:
>>>
>>>> On Thu, Oct 22, 2015 at 7:56 AM, Andreas Schwab <schwab@suse.de> wrote:
>>>>> "H.J. Lu" <hjl.tools@gmail.com> writes:
>>>>>
>>>>>> diff --git a/gas/configure.ac b/gas/configure.ac
>>>>>> index 9002552..ab4dedc 100644
>>>>>> --- a/gas/configure.ac
>>>>>> +++ b/gas/configure.ac
>>>>>> @@ -72,7 +72,7 @@ AC_ARG_ENABLE(compressed_debug_sections,
>>>>>>             AS_HELP_STRING([--enable-compressed-debug-sections={all,gas,none}],
>>>>>>             [compress debug sections by default])],
>>>>>>  [case ,"${enableval}", in
>>>>>> -  *,yes,* | *,all,* | *,gas,*) ac_default_compressed_debug_sections=yes ;;
>>>>>> +  ,yes, | ,all, | *,gas,*) ac_default_compressed_debug_sections=yes ;;
>>>>>
>>>>> What does that fix?  "*,yes,*" is a superset of ",yes,".
>>>>
>>>> It will won't set ac_default_compressed_debug_sections to yes for
>>>
>>> Will or won't?
>>
>> Won't.
>>
>>>> --enable-compressed-debug-sections=not,yes
>>>
>>> What is "not" supposed to do?
>>>
>>
>> No idea.
>
> That doesn't answer my question.

Only

--enable-compressed-debug-sections=yes

means yes, nothing else.


-- 
H.J.