Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
cosminonea opened this issue Oct 8, 2013 · 6 comments
Open

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

cosminonea opened this issue Oct 8, 2013 · 6 comments

Comments

@cosminonea
Copy link

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
Copy link
Member

Looking into it.

@fearthecowboy
Copy link
Member

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
Copy link
Author

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
Copy link
Member

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

Did you mean CreateFile2?

@fearthecowboy
Copy link
Member

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
Copy link

this I think has already been fixed in another fork:
madler/zlib#49

would be handy to get this change made in nuget version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants