Re: CVS commit: src/distrib/sets

[Christoph Egger <Christoph_Egger%gmx.de@localhost>]

Masao Uebayashi wrote:
>> @@ -32,15 +32,16 @@
>>  while getopts L:bxXloa:m:s: ch; do
>>      case ${ch} in
>>      L)
>> -            lists=$(
>> -                    for _list in $( echo ${OPTARG} | tr , ' ' ); do
>> -                            case $_list in
>> -                            base)   echo "${nlists}" ;;
>> -                            x)      echo "${xlists}" ;;
>> -                            ext)    echo "${extlists}" ;;
>> -                            esac
>> -                    done
>> -            )
>> +            save_IFS="${IFS}"
>> +            IFS=,
>> +            for _list in ${OPTARG}; do
>> +                    case $_list in
>> +                    base)   lists="${lists} ${nlists}" ;;
>> +                    x)      lists="${lists} ${xlists}" ;;
>> +                    ext)    lists="${lists} ${extlists}" ;;
>> +                    esac
>> +            done
>> +            IFS="${save_IFS}"
>>              ;;
>>      # backward compat
>>      b)
> 
> Thanks, this looks better.
> 
> Could you elaborate how the old version failed on Mac OS X?

./makeobsolete: line 44: syntax error near unexpected token `)'
./makeobsolete: line 44: `              )'

Christoph