-
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
libdwarf: Add 0.9.0 and 0.9.1 #22629
libdwarf: Add 0.9.0 and 0.9.1 #22629
Conversation
🤖 Beep Boop! This pull request is making changes to 'recipes/libdwarf//'. 👋 @Hopobcn you might be interested. 😉 |
I detected other pull requests that are modifying libdwarf/all 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.
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.
This comment has been minimized.
This comment has been minimized.
Alright, I think this should be good to go now |
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.
Looking good: davea42/libdwarf-code@v0.8.0...v0.9.1
Only small comment.
Co-authored-by: Uilian Ries <uilianries@gmail.com>
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.
Thanks Jeremy :D
recipes/libdwarf/all/conanfile.py
Outdated
@@ -50,6 +53,8 @@ def requirements(self): | |||
if self.options.with_dwarfgen or self.version == "20191104": | |||
self.requires("libelf/0.8.13") | |||
self.requires("zlib/[>=1.2.11 <2]") | |||
if self.version != "20191104" and Version(self.version) >= Version("0.9.0"): |
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.
Looks like we might want to remove this 20191104
version by now if upstream is releasing official ones. Wdyt @uilianries?
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.
Yes, please!!! 👍
@jeremy-rifkin Could you please remove the version 20191104
from config.yml, conandata.yml and update the conanfile.py as well? I know it's not related to this PR, but it would help a lot.
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'd be my pleasure!
This comment has been minimized.
This comment has been minimized.
@jeremy-rifkin Please, keep only libdwarf changed in this PR. The CI not able to build more than one project per PR. |
Thanks, will do |
31c1c52
to
638bb73
Compare
Should updates to libraries depending on libdwarf/20191104 come in separate PRs then? |
Unfortunately yes. Each project requires a separated PR. It should be improved in the future, but for now the CI not capable to process more than one project at same project. |
This comment has been minimized.
This comment has been minimized.
Small bump, is there anything else needed from me to get this merged? |
3a46899
to
a713eb8
Compare
Hopefully now ready to go |
This comment has been minimized.
This comment has been minimized.
I asked to the upstream to update the Zstd cmake targets. Let's see if we can incorporate that patch in the project: davea42/libdwarf-code#225 |
@uilianries I've applied the change in davea42/libdwarf-code#226 for using lowercase |
Conan v1 pipeline ✔️All green in build 11 (
Conan v2 pipeline ✔️
All green in build 11 ( |
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
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.
Thanks for your patience Jeremy:)
Specify library name and version: libdwarf/0.9.0 libdwarf/0.9.1