Re: [Lzip-bug] Version 1.11-rc5 of lzip released

[JonY]

On 8/31/2010 22:55, Antonio Diaz Diaz wrote:
> JonY wrote:
> > Tested and works on win32 with some minor edits.
>
> Thanks for the feedback!
>
> How minor are the edits? Can they be easily fixed like the binary
> stdin/stdout? It seems on OS/2 lzip already compiles without any changes.

Hi,

The hacks are relatively benign. Here are the short inserts to make 
main.cc and lziprecover.cc compile, along with my reasons.

#define S_ISSOCK(X) 0 //File descriptors and sockets are distinct
#define fchmod(X,Y) 0 //FILE and fd could be mapped via fdopen
#define fchown(X,Y,Z) 0 //We don't care for unix perms
#define S_IRGRP 0
#define S_IWGRP 0
#define S_IROTH 0
#define S_IWOTH 0
#define SIGHUP SIGTERM //There is no sighup equivalent

_chmod on win32 isn't very useful, as it only supports read and write 
permissions (I think for the current user only). See 
<http://msdn.microsoft.com/en-us/library/aa246709%28VS.60%29.aspx> for 
more info.

> > It even builds for win64, but I have not tested the 64bit build.
>
> Does "make check" work on windows64?

I will test it soon. I built it accidentally when I forgot to set CXX 
with -m32.