Skip to content
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

Add native GLFW binary builds #792

Merged
merged 15 commits into from
Jan 28, 2022
Merged

Add native GLFW binary builds #792

merged 15 commits into from
Jan 28, 2022

Conversation

roeyskoe
Copy link
Contributor

@roeyskoe roeyskoe commented Jan 27, 2022

Summary of the PR

Self building GLFW for Windows, Mac and Linux, including Mac arm64.
This partially resolves #727.

Related issues, Discord discussions, or proposals

https://discord.com/channels/521092042781229087/587346162802229298/936229014249562172

Further work

Currently the mac libraries cause a security warning, since they are not signed. This needs to be resolved before shipping them.

Silk is also still missing arm64 build for OpenAL, and ImGUI.NET also does not ship with arm natives.

@roeyskoe roeyskoe changed the title Build GLFW with nuke Add native GLFW binary builds Jan 28, 2022
@roeyskoe roeyskoe marked this pull request as ready for review January 28, 2022 09:45
@roeyskoe roeyskoe requested a review from a team as a code owner January 28, 2022 09:46
Copy link
Member

@Perksey Perksey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very happy with this!

@Perksey
Copy link
Member

Perksey commented Jan 28, 2022

Oh also... can you change the submodule reference to point at the latest tagged release instead of presumably the latest develop/master/whatever? The latest release SHA is 7d5a16c.

Thanks!

roeyskoe and others added 2 commits January 28, 2022 15:03
Co-authored-by: Dylan Perks <11160611+Perksey@users.noreply.github.com>
Copy link
Member

@Perksey Perksey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much - you are a godsend! If the PR description is alluding to you possibly doing the same for OpenAL Soft, feel free to do so!

@Perksey Perksey enabled auto-merge (squash) January 28, 2022 13:14
@Perksey Perksey mentioned this pull request Jan 28, 2022
38 tasks
@roeyskoe
Copy link
Contributor Author

I'll look into it probably Sunday/early next week :)

@Perksey Perksey merged commit c31124e into dotnet:main Jan 28, 2022
Perksey added a commit that referenced this pull request Mar 25, 2022
* Start working with native builds

* gh actions now?

* now?

* Actually get the submodules...

* It now actually kinda does something

* Save artifacts for testing

* Not the prettiest, but may actually work

* Maybe now I actually get the artifacts

* Typo

* What about now

* Now lets put files in the right place

* Maybe this is a bit nicer now

* Apply suggestions from code review

Co-authored-by: Dylan Perks <11160611+Perksey@users.noreply.github.com>

* Lock glfw to latest release

Co-authored-by: Dylan Perks <11160611+Perksey@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

.NET 6 arm64 builds
2 participants