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)


Charles Wilson wrote:

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 )
}

Uhm..dunno what I was thinking when I pressed the "send" button, I meant to send the following code:

strip() {
(cd ${instdir} && \
find . -name "*.dll" -or -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.
Doesn't seems "right" to me that the (sometimes normal) absence of DLLs should stop the thing.
You could of course reply that scripts are meant to be adapted to the package...

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.
OK, my ignorance.

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.
Well I actually was talking about what I pasted this time, not that time 0=)

Lapo

--
Lapo 'Raist' Luchini
lapo@lapo.it (PGP & X.509 keys available)
http://www.lapo.it (ICQ UIN: 529796)




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