haskell / win32

Haskell support for the Win32 API
http://hackage.haskell.org/package/Win32
Other
98 stars 62 forks source link

2.10.0.0 tarball is unbuildable #162

Closed phadej closed 3 years ago

phadej commented 3 years ago
PS C:\Users\Oleg Grenrus\Documents\postgresql-libpq> cabal build --constraint='Win32 ^>=2.10'
Resolving dependencies...
Build profile: -w ghc-8.8.4 -O1
In order, the following will be built (use -v for more details):
 - Win32-2.10.0.0 (lib) (requires download & build)
 - postgresql-libpq-0.9.4.3 (lib:postgresql-libpq) (configuration changed)
Downloading  Win32-2.10.0.0
Downloaded   Win32-2.10.0.0
Starting     Win32-2.10.0.0 (lib)
Building     Win32-2.10.0.0 (lib)

Failed to build Win32-2.10.0.0.
Build log ( C:\Users\Oleg
Grenrus\AppData\Roaming\cabal\logs\ghc-8.8.4\Win32-2.10.0.0-3884a87f6116e1eb131fdcd63a8c4f0391e40976.log
):
Preprocessing library for Win32-2.10.0.0..
compiling dist\build\System\Win32\NLS_hsc_make.c failed (exit code 1)
rsp file was: "dist\\build\\System\\Win32\\hsc8915.rsp"
command was: C:\ProgramData\chocolatey\lib\ghc\tools\ghc-8.8.4\lib\../mingw/bin\gcc.exe -c dist\build\System\Win32\NLS_hsc_make.c -o dist\build\System\Win32\NLS_hsc_make.o -D__GLASGOW_HASKELL__=808 -Dmingw32_BUILD_OS=1 -Dx86_64_BUILD_ARCH=1 -Dmingw32_HOST_OS=1 -Dx86_64_HOST_ARCH=1 -Iinclude -IC:\tools\msys64\mingw64\include -Idist\build\include -IC:\tools\msys64\mingw64\include -fno-strict-aliasing -Wall -Idist\build\autogen -Idist\build\global-autogen -include dist\build\autogen\cabal_macros.h -IC:\ProgramData\chocolatey\lib\ghc\tools\ghc-8.8.4\lib\bytestring-0.10.10.1\include -IC:\ProgramData\chocolatey\lib\ghc\tools\ghc-8.8.4\lib\base-4.13.0.0\include -IC:\ProgramData\chocolatey\lib\ghc\tools\ghc-8.8.4\lib\integer-gmp-1.0.2.0\include -IC:\ProgramData\chocolatey\lib\ghc\tools\ghc-8.8.4\lib/include -IC:\ProgramData\chocolatey\lib\ghc\tools\ghc-8.8.4/lib/include/
error: NLS.hsc:56:10: fatal error: winnls_compat.h: No such file or directory
compilation terminated.

When I added

install-includes: "winnls_compat.h", "winnt_compat.h"

and made sdist package, that one build fine.

I'm not sure whether these includes should be install-includes or just includes so I won't make a PR.

Mistuke commented 3 years ago

Fixed in 2.10.1.0, thanks for the report! CI has been modified to check bindist too.