-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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_ttf: and 2.20.2, bump dep sdl, conan v2 fixes #18366
sdl_ttf: and 2.20.2, bump dep sdl, conan v2 fixes #18366
Conversation
This comment has been minimized.
This comment has been minimized.
…15 + add support for 2.20.2 :)
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.
Hooks produced the following warnings for commit c0b8b37sdl_ttf/2.0.15
sdl_ttf/2.0.18
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 5ec9cd1sdl_ttf/2.0.18
sdl_ttf/2.0.15
|
This never built with v2 #15560 (comment) And v2 is not required 🙄
|
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 79d3824sdl_ttf/2.0.18
sdl_ttf/2.0.15
|
Conan v1 pipeline ✔️All green in build 7 (
Conan v2 pipeline ✔️
All green in build 6 ( |
* sdl_ttf: bump dep sdl * patch 3.17 requirements to keep in line with Conan's min supported 3.15 + add support for 2.20.2 :) * Update recipes/sdl_ttf/all/conandata.yml * Update recipes/sdl_ttf/all/conanfile.py * Update recipes/sdl_ttf/all/conanfile.py * Add missing ios frameworks --------- Co-authored-by: Luis Caro Campos <3535649+jcar87@users.noreply.github.com>
* sdl_ttf: bump dep sdl * patch 3.17 requirements to keep in line with Conan's min supported 3.15 + add support for 2.20.2 :) * Update recipes/sdl_ttf/all/conandata.yml * Update recipes/sdl_ttf/all/conanfile.py * Update recipes/sdl_ttf/all/conanfile.py * Add missing ios frameworks --------- Co-authored-by: Luis Caro Campos <3535649+jcar87@users.noreply.github.com>
if Version(self.version) <= "2.0.18" and is_apple_os(self) and self.options.shared: | ||
self.cpp_info.components["_sdl2_ttf"].frameworks = [ | ||
"AppKit", "CoreGraphics", "CoreFoundation", "CoreServices" | ||
] |
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.
It makes no sense, it should be for static only, not shared.
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.
Please open an issue with more information :) If the hooks are incorrect #18366 (comment) please open an issue there so we can track it 🙏
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.
Hook is correct, but your interpretation of its result is incorrect.
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.
This hook just says that shared lib links to some system libs or frameworks, so it means that these system libs/frameworks must be added if static. This hook has always been an indirect way to find system libs or frameworks to add if static (for shared it doesn't matter since it's already linked obviously).
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.
I'd argue the hooks are bad for providing misleading information
Library './lib/libSDL2_ttf.dylib' links to system library
implies the shared one needs it, if that incorrect -- then the messages are unclear -- it should have included the settings and options to be more concise :)
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.
Feel free to update message hook if you think it's misleading.
Instead of:
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libSDL2_ttf.dylib' links to system library 'CoreServices' but it is not in cpp_info.frameworks.
it could be:
post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libSDL2_ttf.dylib' links to system library 'CoreServices', therefore 'CoreServices' should be in cpp_info.frameworks if static, but it's missing.
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.
Done, thanks for the suggestion!
Specify library name and version: lib/1.0