This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: standard build script (method 2)



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



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]