-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
qt/6: Bump openssl version #10050
qt/6: Bump openssl version #10050
Conversation
I detected other pull requests that are modifying qt/6.x.x recipe: This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: ericLemanissier <ericLemanissier@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
I've checked it and We need to know where does this packageID for |
regenerate binaries for conan-io#10050
regenerate binaries for #10050
This comment has been minimized.
This comment has been minimized.
Co-authored-by: ericLemanissier <ericLemanissier@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
you can also bump pcre2, glib, icu, libpq, sqlite3 & expat. |
This comment was marked as outdated.
This comment was marked as outdated.
disable gstreamer by default
Failure in build 11 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
There is this error on the log:
I've no idea how to solve this. If someone want to continue this PR which seems to be long waited for many people please do because I can't manage to dedicate more time to it. |
if self.options.with_pcre2: | ||
self.requires("pcre2/10.37") | ||
self.requires("pcre2/10.39") |
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.
self.requires("pcre2/10.39") | |
self.requires("pcre2/10.37") # needs to be < 10.38 |
Woopsie, we already went through this one earlier in this PR 😊
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.
Due to PCRE2Project/pcre2@2410fbe
I've left a comment few weeks ago to warn against this breaking change in static lib name for Visual Studio: PCRE2Project/pcre2@2410fbe#commitcomment-68176307
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.
So it could be fixed in qt, the comment is misleading (it seems to say that pcre2 >= 10.38 is not compatible with qt, which is not true).
Instead of https://github.com/qt/qtbase/blob/v6.2.4/cmake/FindWrapSystemPCRE2.cmake#L27-L32:
find_library(PCRE2_LIBRARY_RELEASE
NAMES pcre2-16 pcre2-16-static
HINTS ${PC_PCRE2_LIBDIR})
find_library(PCRE2_LIBRARY_DEBUG
NAMES pcre2-16d pcre2-16-staticd pcre2-16
HINTS ${PC_PCRE2_LIBDIR})
@MartinDelille I integrated all your changes in #10261 so you can close this PR if you want. |
Thanks @ericLemanissier ! I close it in favor of the promising #10261 then! 😄 |
Specify library name and version: qt/6.x.x
This is also a good place to share with all of us why you are submitting this PR (specially if it is a new addition to ConanCenter): is it a dependency of other libraries you want to package? Are you the author of the library? Thanks!
conan-center hook activated.