-
Notifications
You must be signed in to change notification settings - Fork 2k
libpqxx: add version 6.4.8, support conan v2, remove older versions #14371
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
Conversation
Try to close #14117. |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 753ef63libpqxx/7.7.3
libpqxx/7.3.2
libpqxx/7.7.4
libpqxx/7.4.2
libpqxx/7.6.1
libpqxx/7.6.0
libpqxx/7.7.2
libpqxx/7.7.0
libpqxx/7.0.7
libpqxx/7.1.2
libpqxx/7.5.3
libpqxx/7.2.1
libpqxx/6.4.8
|
This comment has been minimized.
This comment has been minimized.
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.
LGTM
recipes/libpqxx/all/conanfile.py
Outdated
else: | ||
check_min_vs(self, 192) | ||
|
||
if is_msvc(self) and self.options.shared and self.settings.compiler.runtime == "MTd": |
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.
compiler.runtime
is specific to Visual Studio
. Can you use conan.tools.microsoft.msvc_runtime_flag() which supports both Visual Studio
and msvc
to check against the MTd
flag here?
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.
@jwillikers
Thank you for your comments!
I fixed it.
Let's see it works well.
Conan v1 pipelineAll green in build 4 (
|
Specify library name and version: libpqxx/*