-
-
Notifications
You must be signed in to change notification settings - Fork 441
A bunch of improvements to native library builds #2049
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
Conversation
a427837 to
fc5dacb
Compare
|
Just going to kick off the builds for the purposes of review - just close the PRs as they're opened, I'm only interested in the checkmarks. |
Perksey
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing like a nice glass of Bailey's to go along with an evening code review (it's a good thing I found no comments because I drank the last bit and import prices in Texas are ridiculous)
@Beyley PTAL
27c4f51 to
e9d8ec6
Compare
These will mainly be used in CI workflows, so they're specific to Ubuntu 18.04 and 22.04.
Also clean up the build script.
Also clean up the build script.
Perksey
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest changes are fine provided @Beyley agrees
The sndio backend takes a hard dependency on libsndio.so, unlike most other backends which load libraries dynamically. We don't want our libopenal.so to require users to install sndio.
This allows the build to pick them up and enable support.
This package is not multiarch-compatible in 22.04, so prioritize amd64.
This package is not multiarch-compatible in 22.04, so prioritize amd64.
…Fri Mar 15 00:10:50 PDT 2024; root:xnu-10063.101.17~1/RELEASE_ARM64_VMAPPLE
…P Sat Mar 9 04:50:38 UTC 2024
…ntu SMP Sat Mar 9 04:50:38 UTC 2024
Update OpenALSoft binaries
Update SDL2 binaries
Beyley
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only thing left is just a missing comment, add that and ale li pona tawa mi, really nice work!
|
Maybe it's worth factoring out the Ubuntu extras as well into a bash script? |
The proper way to handle the duplication of logic in the native workflows is to switch them over to GitHub's reusable workflows feature, but that's a project for another day. |
Beyley
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pona
|
Force merging because CLA is sleeping |
-O3.linux-x86support (was never officially supported by .NET).Contributes to #727.
Closes #2015.
Closes #2016.