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: for generic-build-script


Hello Igor,

>>>>> -    find ${instdir}${prefix}/share/info -name "*.info" | xargs gzip -q ; \
>>>>> +    find ${instdir}${prefix}/share/info -name "*" | xargs gzip -q ; \
>>>> -name "*" is a no-op.  You can just remove it.
>>> Yep.  I think what was intended here was "-type f".
>> Yes, of course.
> Actually, given your explanation in the previous message, why wouldn't
> '-name "*.info*"' suffice?

Yes, that would do it too.


[...]

>> +    ( cd ${instdir}${prefix}/share/man && gzip -fq9 */* ) ;\
> Are you sure gzip will handle subdirectories correctly?  I have to look at
> gzip's manpage and info pages carefully...

It works for me(tm).


>>    fi && \
>>    templist="" && \
>>    for f in ${install_docs} ; do \

> One more thing: shouldn't the separators in the commands be '&&' instead
> of ';'?

I have not tested it, but it is included in
if ; then
 ...
fi
I guessed that ; at the end of the 'then' part is correct , and
actually I only changed the 'find ... gzip ...' part between 'then'
and the semikolon.


Gerrit
-- 
=^..^=



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