This is the mail archive of the cygwin-patches@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: [PATCH] Checking integrity of installed packages in cygcheck


On Sat, Aug 09, 2003 at 03:18:00PM -0400, Charles Wilson wrote:
>Christopher Faylor wrote:
>>I'll check this in but it would be nice if (WBNI) this used a mingw gzip
>>library rather than calling gzip directly.  That's a fair amount of
>>work but I could resurrect the zlib library in winsup if necessary.
>>
>>I wonder why setup is using gzip rather than bzip2 for the package files...
>
>the setup tree contains its own copies of the zlib and bzlib trees; 
>thue, they are compiled under the same runtime that setup is.  If setup 
>is a 'mingw' app, then so are the internal, statically linked libz and 
>bz2lib.

Yes, of course, I know about setup's zlib and bz2lib (having imported
both of them <or maybe DJ imported zlib.  Don't remember exactly.>).  I
kept the bz2lib around in winsup after moving setup out of winsup for
its eventual use by files in utils.  As I mentioned, I would be willing
to resurrect zlib also.

This doesn't explain why setup is using gzip over bzip2 since both are
available and bzip2 generally (but not always) provides superior
compression.

(I see that Robert indicates that this is historical but I thought that
the compression of the package files in /etc/setup postdates the
availability of bz2 compression in setup.  Guess I was wrong.)

>I imagine that the reason Igor used popen and zcat is simply that it was 
>easier than directly interfacing to the library.  Perhaps that issue 
>could be addressed in a later patch (along the lines of the compress_gz 
>class, which also provides uncompression capabilities?)

Yes.  It would have to be a later patch since the current implementation
is already checked in.

As I said, "That's a fair amount of work but I could resurrect the zlib
library in winsup if necessary."

cgf


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