-
-
Notifications
You must be signed in to change notification settings - Fork 203
VCPKG And Various Fixes #1579
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
Open
marauder2k7
wants to merge
44
commits into
TorqueGameEngines:development
Choose a base branch
from
marauder2k9-torque:VCPKG-Test
base: development
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
VCPKG And Various Fixes #1579
marauder2k7
wants to merge
44
commits into
TorqueGameEngines:development
from
marauder2k9-torque:VCPKG-Test
Conversation
This file contains hidden or 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
remove openal configurations. vcpkg does this for us
openal needs to be built as dynamic add an arm64-osx-11 triplet to build for macosx may need an x64 couterpart next up add linux.
remove space from my projects folder
since vcpkg sets up openal-soft for us we no longer have to set certain defines that control whether certain features are okay per platform. We just check ALC_EXT_EFX and load the functions as needed. More ground work to be added to this and since vcpkg is going to be rolled in this is the most logical place for it
fix defines for mac and linux
SFX System now loads up in a similar manner to the GFX System. SFXProviders built on launch, sfxdevice chosen on launch. SFXAL Cleanup no more need for the mOpenal table to be shared across classes.
mac teardown fix, mSearchEdit being null and calling deleteObject was causing issues.
this should make our libs static as it will find the ports we define for them
unexpected REF in pcre portfile
silence warnings
TORQUE_LIB_ROOT_DIRECTORY is not set at this point
profile safeties
clean vcpkg after build, no longer need the packages after everything is installed
231d0bf to
60f3bd6
Compare
stop copying the .in and .cmake files
seperate vcpkg libs from torque_link_libraries so we can put those back into a folder in the solution test of linking dx compiler
woops
move sync back project outside installTemplate macro so it remains in the project VCPKG find_package calls should have CONFIG
the manifest file allows us to specify specific versions of libraries
Contributor
|
to be very, very clear: while the idea has a great deal of merit, and we look forward to the kinks getting knocked off, there's enough vetting that needs to go into this one it's not going to make the 4.1 cutoff. |
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.
Added VCPKG to run the build on each platform.
VCPKG finds dependencies and sets up the environment for our own dependencies to build properly.
Other Fixes:
SFX System now integrates providers in a similar way the GFX system integrates adapters SFXDevice will setup automatically before the soundAssets are loaded meaning the buffers that need to be loaded should actually succeed.
MacOSX had an issue on teardown where it would try and free objects that didnt exist. This may have been causing issues on other platforms as well issue was in GuiPopUpCtrlEx
OpenAL Loading now uses system dll's as openal-soft is compiled using the al_router.
Templates can now be installed without copying their .in files to the output directory
===================================================
LibSndFile operates with an LGPL license and needs to be linked dynamically
Assimp now has its dependencies compiled through vcpkg
Faster build times since we are not compiling all the dependencies over and over again, vcpkg compiles them once.
Universal build now works on Mac