17cupsofcoffee / tetra

🎮 A simple 2D game framework written in Rust
MIT License
909 stars 63 forks source link

Update sdl2 requirement from 0.34.0 to 0.35.0 #286

Closed dependabot[bot] closed 2 years ago

dependabot[bot] commented 2 years ago

Updates the requirements on sdl2 to permit the latest version.

Changelog

Sourced from sdl2's changelog.

v0.35.0

  • BREAKING CHANGE Update sdl2-sys/sdl_bindings.rs to use enums instead of consts. If you were using sdl2-sys's enum variants directly in your project, you may be affected. If you only used sdl2 calls, there should not be any problems.

[PR #1138](Rust-SDL2/rust-sdl2#1138) Added binding for SDL_OpenURL

[PR #1150](Rust-SDL2/rust-sdl2#1150) Do not download SDL2 sources when using bundled feature

[PR #1112](Rust-SDL2/rust-sdl2#1112) Add wrapper functions for SDL_RenderSetIntegerScale and SDL_RenderGetIntegerScale

[PR #1156](Rust-SDL2/rust-sdl2#1156) Maybe breaking change: new variants to enum GameController, and prevent panic with unrecognized buttons.

[PR #1153](Rust-SDL2/rust-sdl2#1153) SDL_GL_GetCurrentContext and SDL_RenderFlush added.

[PR #1131](Rust-SDL2/rust-sdl2#1131) Added Sensor API.

v0.34.5

[PR #1100](Rust-SDL2/rust-sdl2#1100) Added binding for SDL_GetDisplayUsableBounds

[PR #1102](Rust-SDL2/rust-sdl2#1102) Correctly se linux and macSO built libraries when using bundled without static-link.

[PR #1098](Rust-SDL2/rust-sdl2#1098) Fix potential heap corruption when using AudioCVT::convert

[PR #1088](Rust-SDL2/rust-sdl2#1088) Rollback of PR #1081: Broke dynamic linking on Windows

Various fixes to CI.

v0.34.4

[PR #1086](Rust-SDL2/rust-sdl2#1086) Update bundled to use SDL2 2.0.14

[PR #1033](Rust-SDL2/rust-sdl2#1033) Changed signature of TimerSubsystem::ticks to accept &self.

[PR #1057](Rust-SDL2/rust-sdl2#1057): fix memory safety bug in set_error

[PR #1081](Rust-SDL2/rust-sdl2#1081): Allow bundled build to be built in debug mode. Fixes issue when linking binary with mixed debug+release CRT dependencies.

[PR #1080](Rust-SDL2/rust-sdl2#1080): Fix line endings of patches to lf so patching of sources works on Windows.

[PR #1031](Rust-SDL2/rust-sdl2#1031): Add patch to fix metal detection (https://bugzilla.libsdl.org/show_bug.cgi?id=4988)

v0.34.3

[PR #1027](Rust-SDL2/rust-sdl2#1027): upgrade "bundled" version of SDL2 to 2.0.12 [PR #1020](Rust-SDL2/rust-sdl2#1020): revert undefined behavior introduction. Commit: Fix clippy issue Commit: Temporarily suppress dangerous clippy hints PR#1014: add load_texture_bytes interface for load texture from buffer

... (truncated)

Commits


Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Tairesh commented 2 years ago

Added binding for SDL_OpenURL

Wow, this can be useful

17cupsofcoffee commented 2 years ago

@Tairesh That does seem handy 😄 Though I need to be kinda careful about using functionality that's relatively new in SDL2, I've had issues before where old versions would fail to link properly to the sdl2 crate. Might have just been me doing something wrong though, I'll have to experiement some more.

dependabot[bot] commented 2 years ago

Looks like sdl2 is up-to-date now, so this is no longer needed.

17cupsofcoffee commented 2 years ago

I ran into the linking issue I mentioned after updating the dependency - even though I don't use SDL_OpenURL anywhere in my code, it's in the sdl2-sys bindings now, so it tries to link the symbol.

That said, this made me realize that the issue is Windows-specific, because the CI builds (which run on Ubuntu) are linking against SDL2 2.0.10 absolutely fine 🙂 I can live with that, because updating SDL2 is really easy on Windows (just have to drop the new files into the project).

My worry was that using new stuff would cause builds to stop linking on LTS versions of Ubuntu (which won't get new versions of SDL2 in their package manager), but that doesn't seem to be the case.

So TL;DR: we don't need to be afraid of using new SDL2 functions in Tetra 😄