-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
sdl_mixer: add v2.8.0 as a new subdir #21782
Conversation
8bcd592
to
b8b1f0e
Compare
This comment has been minimized.
This comment has been minimized.
Rebuilding the missing dependency now :) |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Johnny Jazeix <be__good@hotmail.com>
This comment has been minimized.
This comment has been minimized.
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
fee92f8
to
ac5d32d
Compare
This comment has been minimized.
This comment has been minimized.
Please edit the pull request title regarding the version, so that the merge commit message will be correct |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ✔️All green in build 17 (
Conan v2 pipeline ✔️
All green in build 17 (
|
@RubenRBS Can we please get a review? 🙂 |
@@ -0,0 +1,8 @@ | |||
cmake_minimum_required(VERSION 3.15) |
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.
test_v1_package are no longer necessary, it is better to not add them, specially for new recipes (this under cmake
is a new recipe)
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.
No need to change it now, but just FYI. Less is more, specially when the backlog in ConanCenter is so big.
* sdl_mixer: add v2.6.3 as a new subdir * sdl_mixer: fix license install * sdl_mixer: remove unnecessary install dirs * sdl-mixer: add GitHub URL * sdl-mixer: set SDL2MIXER_DEBUG_POSTFIX * sdl_mixer: fix typo Co-authored-by: Johnny Jazeix <be__good@hotmail.com> * sdl_mixer: fix typo in .set_property() Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com> * sdl_mixer: use a more specific include * sdl_mixer: add a -static CMake target alias * sdl_mixer: rmdir cmake * sdl_mixer: fix MSVC static lib name * sdl_mixer: bump to v2.8.0 * sdl_mixer: drmp3 has been replaced with minimp3 * sdl_mixer: update options * sdl_mixer: disable debug postfix * sdl_mixer: add missing frameworks * sdl_mixer: review changes * sdl_mixer: revert flac version bump --------- Co-authored-by: Johnny Jazeix <be__good@hotmail.com> Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
* sdl_mixer: add v2.6.3 as a new subdir * sdl_mixer: fix license install * sdl_mixer: remove unnecessary install dirs * sdl-mixer: add GitHub URL * sdl-mixer: set SDL2MIXER_DEBUG_POSTFIX * sdl_mixer: fix typo Co-authored-by: Johnny Jazeix <be__good@hotmail.com> * sdl_mixer: fix typo in .set_property() Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com> * sdl_mixer: use a more specific include * sdl_mixer: add a -static CMake target alias * sdl_mixer: rmdir cmake * sdl_mixer: fix MSVC static lib name * sdl_mixer: bump to v2.8.0 * sdl_mixer: drmp3 has been replaced with minimp3 * sdl_mixer: update options * sdl_mixer: disable debug postfix * sdl_mixer: add missing frameworks * sdl_mixer: review changes * sdl_mixer: revert flac version bump --------- Co-authored-by: Johnny Jazeix <be__good@hotmail.com> Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
The existing recipe is quite outdated. The project has switched over to using CMake.
This also effectively migrates the package to Conan v2.