-
Notifications
You must be signed in to change notification settings - Fork 30
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
attempt to cross-compile for Win32 #33
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
Birch-san
commented
Jan 3, 2022
•
edited
Loading
edited
- i686 Standalone: works
- i686 VST2: works in FL Studio
- i686 VST3: "error", "!isValid" in FL Studio
- x86_64 Standalone: works
- x86_64 VST2: works in FL Studio
- x86_64 VST3: works in FL Studio
…cmake throws away my variables.
… try building fluidsynth in Release mode.
…d, since there are missing dependencies.
Birch-san
force-pushed
the
win32-cross-compile
branch
from
January 5, 2022 23:02
7967d03
to
b9f9ccf
Compare
…try to install libintl. revert to mainline fluidsynth + manual installation (to avoid potential failure to link fluidsynth targets that we don't need)
…t minimum C++ standard. go back to manual installation of libfluidsynth, so that we don't need to maintain a fork with bespoke link flags.
…-config's compile advice). make up for absence of pkg-config file expressing dependency on libiconv.
…wind) statically.
…flags there (location of libomp, and -pthread). neither helped us (we've now eliminated libomp because OpenMP doesn't support static linking, and the -pthread flag attempted a dynamic link anyway). use CMAKE_EXE_LINKER_FLAGS to pass in winpthread library. delete out-of-date sandboxing pessimism (since I already implemented security-scoped tokens).
…contingent on whether SDK headers are supplied. enable x86 and Release builds.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.