standard build script (method 2)

[Charles Wilson]

Lapo Luchini wrote:

>> strip() {
>>   (cd ${instdir} && \
>>   find . -name "*.dll" | xargs strip > /dev/null 2>&1
>>   find . -name "*.exe" | xargs strip > /dev/null 2>&1 )
>> }
> 
> 
> 
> Shouldn't that be changed to the following?
> 
> strip() {
>  (cd ${instdir} && \
>  find . -name "*.dll" | xargs strip > /dev/null 2>&1 && \
>  find . -name "*.exe" | xargs strip > /dev/null 2>&1 )
> }


No.  If there are no *.dll files, then xargs fails -- which would mean 
that the exe's don't get stripped.


> In some cases it gives me problems as the two lines aren't separated by 
> anything (&& or ;)


Yes, they are separated -- by an un-backslashed newline.  That's plenty.

> and moreover doing it all in a single line is a 
> little more optimized.


It's possible to optimize into non-functionality, which is what '&& \' 
would do.

--Chuck