Re: [PATCH] Unbreak `make install' in leim/ when leim's ${INSTALLDIR} doesn't exist

[Giorgos Keramidas]

On 2006-10-12 14:47, Andreas Schwab <address@hidden> wrote:
>Giorgos Keramidas <address@hidden> writes:
>> It seems to include this part:
>>
>> +       if [ ! -d ${INSTALLDIR} ] ; then \
>> +          ${srcdir}/${dot}${dot}/mkinstalldirs ${INSTALLDIR}; \
>> +       else true; fi
>>
>> We don't really *need* to have an else branch in all statements.  We can
>> write this as:
>>
>> +       if [ ! -d ${INSTALLDIR} ] ; then \
>> +          ${srcdir}/${dot}${dot}/mkinstalldirs ${INSTALLDIR}; \
>> +       fi
> 
> Except that some old shells will cause the whole command to fail if the
> condition fails.

I see.  I didn't know about this.  Thanks for the clarification :)