-
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
Added 2.1 liburing version #7266
Conversation
This comment has been minimized.
This comment has been minimized.
conan-io/conan#7208 |
This reverts commit 4250b69.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
About 2.1 build fail: |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Due to restriction of using platform python module in configure and validate methods. It is impossible for now to build this version on hosts with glibc 2.26 and less
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.
Did you know that it is not this easy to find gcc5 nowadays? |
Hello @TheSalvator do you really need all those versions? We try avoiding old project versions to save some CI resource, and also to keep the maintenance cleaner (less versions, less code). |
Sorry, I didn't know this, I'll comment out old version. Still there is a new version of liburing 2.1 and it cannot be built on hosts with glibc < 2.27. I think I need help to create some workaround since I cannot use platform module in configure |
I detected other pull requests that are modifying liburing/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. |
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.
There are also some minor comments from other reviewers.
recipes/liburing/all/patches/0001-liburing-2.1-memfd-create.patch
Outdated
Show resolved
Hide resolved
recipes/liburing/all/patches/0001-liburing-2.1-memfd-create.patch
Outdated
Show resolved
Hide resolved
recipes/liburing/all/conanfile.py
Outdated
def requirements(self): | ||
self.requires("linux-headers-generic/5.13.9") | ||
def build_requirements(self): | ||
self.build_requires("linux-headers-generic/5.13.9") |
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, what is the purpose here? Probably using one profile it doesn't make any difference, but is it needed (does it work) if using two profiles?
This comment has been minimized.
This comment has been minimized.
This reverts commit 574a182.
This comment has been minimized.
This comment has been minimized.
Removed unused import Co-authored-by: Uilian Ries <uilianries@gmail.com>
Hey guys, can you review changes one more time? |
@@ -1,5 +1,5 @@ | ||
Name: liburing | ||
-Version: 2.0 | ||
+Version: 2.1 |
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.
Nice, but not really needed for this 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.
It doesn't affect anything, if you insist, I'll remove 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.
You can leave it. It was just a remark.
@jgsogo |
* Added 0.1 0.2 0.3 0.4 0.5 0.6 and 2.1 liburing versions * Revert "(conan-io#7208) liburing: remove call to `platform` in validate" This reverts commit 4250b69. * Removed linux version check * Added statx fix patch to 0.5 version * Temporary disabled 2.1 Due to restriction of using platform python module in configure and validate methods. It is impossible for now to build this version on hosts with glibc 2.26 and less * Updated patch for liburing-0.5 * Added statx patches for 0.3 0.4 and 0.5 * Commented out old liburing versions * Revert "Temporary disabled 2.1" This reverts commit 6685ac1. * Disabled build for gcc5 and less * Added glibc version check through ldd * Removed old versions * Protected get_glibc_version method * Reverted test package from 568d0c6 * Improve glibc detection Signed-off-by: Uilian Ries <uilianries@gmail.com> * Update recipes/liburing/all/conanfile.py Co-authored-by: Ilya Kazakov <31013302+TheSalvator@users.noreply.github.com> * Added patch to deal with missing memfd_create * Removed glibc check * Revert "Revert "(conan-io#7208) liburing: remove call to `platform` in validate"" This reverts commit 260a8eb. * Moved linux headers package to build requires * (conan-io#7266) Updated patch file for liburing 2.1 * Revert "Moved linux headers package to build requires" This reverts commit 574a182. * Update recipes/liburing/all/conanfile.py Removed unused import Co-authored-by: Uilian Ries <uilianries@gmail.com> Co-authored-by: Uilian Ries <uilianries@gmail.com>
liburing/0.1
liburing/0.2
liburing/0.3
liburing/0.4
liburing/0.5
liburing/0.6
liburing/2.1
conan-center hook activated.