JuliaMultimedia / SimpleDirectMediaLayer.jl

SDL2
MIT License
79 stars 17 forks source link

Switch from BinDeps to BinaryProvider (BinDeps2) #39

Closed NHDaly closed 4 years ago

NHDaly commented 4 years ago

Closes https://github.com/jonathanBieler/SimpleDirectMediaLayer.jl/issues/38

Enable the work that @jonathanBieler did to set up BinaryBuilder/BinaryProvider for this repo! :) This PR is just the last 1% of the work @jonathanBieler did to set everything up and get it working. Thanks for that! haha sorry to swoop-in at the last minute and create the PR; i was just excited to see it turned on. :)

Set up to pull the binaries from https://github.com/jonathanBieler/SDL2Builder by just downloading the generated build.jl and replacing it in this repo.

jonathanBieler commented 4 years ago

Thanks, I need to make builders for libSDL2_ttf and libSDL2_mixer before we can switch though, otherwise it's gonna break people code.

NHDaly commented 4 years ago

Ah, true, agreed. Good point! :)

I'd say you can probably just combine them all in one builder. I don't see much harm in that. If you want you can open multiple separate builder repos as well, it just makes maintenance a bit harder of course.

giordano commented 4 years ago

Note that SDL2 is already in Yggdrasil and it has been built with X11 support, too: https://github.com/JuliaPackaging/Yggdrasil/blob/master/S/SDL2/build_tarballs.jl. We also have Wayland for Linux and we could add other dependencies, if needed. I can produce a build.jl for this, if you want.

jonathanBieler commented 4 years ago

That would be great yes, I'm definitively not an expert at this (although I start to understand a bit how the system works).

giordano commented 4 years ago

I've opened a PR at https://github.com/NHDaly/SimpleDirectMediaLayer.jl/pull/5 against the branch of this PR

NHDaly commented 4 years ago

Woah, thanks @giordano! :)

Woahhh, reading about Yggdrasil, it sounds really crazy! Neat! Lol i have learned from google that it is an ancient norse tree? 😂

The BinaryBuilder stuff has changed a lot since I last looked. Thanks for pointing this out!

jonathanBieler commented 4 years ago

@NHDaly Have you seen the blog post here ? seems pretty cool indeed.

Otherwise I think you need to merge the PR on your branch.

NHDaly commented 4 years ago

Yep, thanks. I'm waiting for the build to be fixed over there. :)

aviks commented 4 years ago

There are now jll packages for SDL2, SDL2_ttf, SDL2_image and SDL2_mixer. How do we want to proceed here? Make a 1.3+release with these changes? I'm not sure how easy it will be to port these jll packages to 1.0-1.2.

giordano commented 4 years ago

I'm not sure how easy it will be to port these jll packages to 1.0-1.2.

In my experience, for libraries having many dependencies, using BinaryProvider is quite a pain. I'm so glad JLL packages handle dependencies in a much simpler way.

aviks commented 4 years ago

This can now be closed.