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

Upgrade deps #347

Merged
merged 6 commits into from
May 17, 2024
Merged

Upgrade deps #347

merged 6 commits into from
May 17, 2024

Conversation

archbirdplus
Copy link
Contributor

@archbirdplus archbirdplus commented May 13, 2024

  • Fetch target-specific cubyz_deps.
  • Add option to build from local cubyz_deps.
  • Fetch headers separately from the binaries.
  • Link to official releases of the cubyz_deps.

Tangentially related to this PR.

@archbirdplus archbirdplus marked this pull request as ready for review May 15, 2024 21:22
@archbirdplus
Copy link
Contributor Author

Looks like it's ready, only thing is that these urls in build.zig.zon need to be set, which is up to you.

@IntegratedQuantum
Copy link
Member

I linked the release and made it work on my gnu linux.

It would be great if you could test if the linked macos libraries work on your machine.

@archbirdplus
Copy link
Contributor Author

Looks like you didn't use my fork of GLFW. GLFW fails with X11: Failed to load Xlib because it doesn't attempt to dlopen the appropriate file.
But how did you cross-compile to MacOS? Did you use the MacOSX SDK headers repo?

@IntegratedQuantum
Copy link
Member

No, I used the master branch, which has your PR merged.

For cross compilation I manually fetched all the necessary headers from the SDK headers repo.

@archbirdplus
Copy link
Contributor Author

Master branch does not have my PR merged. In the meantime, use mine: https://github.com/archbirdplus/glfw/releases/tag/Mac-x11

@IntegratedQuantum
Copy link
Member

Oh, i didn't notice you made a PR in the glfw repo, I thought you meant the PR you made to Cubyz-libs.

I'll try building with you glfw version then.

@IntegratedQuantum
Copy link
Member

Alright I updated it. Can you try again?

@archbirdplus
Copy link
Contributor Author

Compiles and runs. Will work out the bugs later

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.

2 participants