-
Notifications
You must be signed in to change notification settings - Fork 67
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
Cannot find source file: wavpackdll/wavpackdll.rc #116
Comments
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. |
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 |
@dbry yes, given that the |
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
The text was updated successfully, but these errors were encountered: