Open jonaski opened 2 years ago
Never mind the second error, it's caused by gcc being set as ASM compiler
Hi, and thanks for reporting this! Unfortunately, the Cmake stuff is one of the areas I know very little about and from the output you posted it looks like the system is still trying to do a *nix-style build rather than MSVC. Have you tried using the MSVC solution file directly, or is there a reason that doesn't work for you?
@evpobr can you provide any more insight into this? Thanks!
Hi. Ok, I will take a look.
Cannot reproduce bug:
Are you running the configuration from VS command prompt?
I also advise, if possible, always use package managers. For example Vcpkg:
wavpackdll/wavpackdll.rc is missing from wavpack-5.4.0.tar.xz, git works. The linking error was caused by a MinGW installation in PATH, so that's not a bug.
So it's ok now?
Yeah but you might want to include wavpackdll.rc in the tarball
@jonaski Thanks for reporting this!
@evpobr @SoapGentoo So is the correct and complete way to fix this to simply add wavpackdll/wavpackdll.rc
to the EXTRA_DIST
section of Makefile.am
? It looks like packages generated from the Cmake
system already include everything.
@dbry yes, given that the .rc
is not part of any native Automake rule, adding it to EXTRA_DIST
should pick it up for the tarball
I'm getting the following error when compiling on Windows with MSVC (Visual Studio 2019).
If I remove "wavpackdll/wavpackdll.rc" from CMakeLists.txt:
sed -i '/wavpackdll.rc/d' wavpack-5.4.0/CMakeLists.txt
CMake configuration completes and I end up with a linking error:
Removing libwavpack.def from the link command works, but there is no .lib file. (Or removing it from sources here: https://github.com/dbry/WavPack/blob/master/CMakeLists.txt#L337)
I have tried the latest code from git, same issue. Samme issue in GitHub Actions CI.
Another issue is that wavpack.h is installed directly in include instead of include/wavpack