search

coapp-packages / zlib

CoApp package for zlib
coapp.org
7 stars 5 forks source link

The nuget package on nuget.org requires windows 8 #5

Open cosminonea opened 11 years ago

cosminonea commented 11 years ago

http://www.nuget.org/packages/zlib/

The nuget package was created on windows 8 or server 2012 and it fails to load on previous versions.

CopyFile2 method in kernel32.dll is not available on previous versions of windows and zlib requires it.

fearthecowboy commented 11 years ago

Looking into it.

fearthecowboy commented 11 years ago

Which variant are you seeing this with? I'm not seeing any imports that even reference copyfile2. Hmm. Unless it's the VC12 CRT that's doing that.

cosminonea commented 11 years ago

zlib.redist.1.2.8.6\build\native\bin\v110\Win32\Debug\dynamic\cdecl\zlib.dll zlib.redist.1.2.8.6\build\native\bin\v110\Win32\Debug\dynamic\stdcall\zlib.dll

Try looking at these on a windows 7/ 2008 machine. It is CreateFile2 that is missing not CopyFile2, sorry.

fearthecowboy commented 11 years ago

Ah, thanks, that narrows it down for me. Hmm.

Did you mean CreateFile2?

fearthecowboy commented 11 years ago

Ah, I see what's going on. Hmm. IOWin32 has support for WINRT and defines that by default.

We're going to have to use an additional set of pivots to support WinRT separately from windows.

izzyonstage commented 10 years ago

this I think has already been fixed in another fork: https://github.com/madler/zlib/issues/49

would be handy to get this change made in nuget version