-
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
add tllist library #8588
add tllist library #8588
Conversation
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.
Sadly I do not have access to logs :( Don't know what is happening. |
This comment has been minimized.
This comment has been minimized.
Hi! This is failing at the |
Oh! You are requiring Conan v1.44 and we are still using 1.43.1 here in CCI
We should propagate this error (at this step) to the user... we try to hide everything unexpected just in case, but for sure this step can raise this way and the error needs to be in the output. |
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.
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.
Thank you for your contribution. Please, consider aacebedo#1
self.build_requires("ninja/[>=1.10.0]") | ||
|
||
def build(self): | ||
meson = Meson(self) |
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.
meson-based test package, something unusual, but still welcomed!
settings = "os", "compiler", "build_type", "arch" | ||
|
||
def build_requirements(self): | ||
self.build_requires("pkgconf/[>=1.7.4]") |
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.
we don't use version ranges, but as it's a test_package, should be harmless?
self.build_requires("pkgconf/[>=1.7.4]") | ||
self.build_requires("meson/[>=0.60.0]") | ||
self.build_requires("ninja/[>=1.10.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.
please don't use version ranges 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.
version ranges are not allowed in CCI
https://github.com/conan-io/conan-center-index/blob/master/docs/faqs.md#why-are-version-ranges-not-allowed
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This comment has been minimized.
This comment has been minimized.
All green in build 6 (
|
I detected other pull requests that are modifying tllist/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. |
Specify library name and version: tllist/1.0.5
Not the author of the library but a project I am working on needs it.
conan-center hook activated.