-
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 #10763
Add tllist #10763
Conversation
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
This comment has been minimized.
This comment has been minimized.
recipes/tllist/all/conanfile.py
Outdated
def validate(self): | ||
if self.settings.os != "Linux": | ||
raise ConanInvalidConfiguration("Only Linux supported") |
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.
What is specific to Linux in this header? It works like a charm on macOS. I guess also on Windows.
Co-authored-by: SpaceIm <[email protected]>
Co-authored-by: SpaceIm <[email protected]>
Co-authored-by: SpaceIm <[email protected]>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
recipes/tllist/all/conanfile.py
Outdated
# FIXME: Fails in test_pacakge with the Windows configuration | ||
if self.settings.os == "Windows": | ||
raise ConanInvalidConfiguration("Windows not supported") |
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.
only Visual Studio
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.
@@ -0,0 +1,40 @@ | |||
from conan import ConanFile |
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.
what is the requirement in terms of conan version?
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.
https://github.com/conan-io/conan/releases/tag/1.43.0 from 1.43.0
I am updating the required version right now 🏃 Thanks! 🦅 👁️
This comment has been minimized.
This comment has been minimized.
Co-authored-by: SpaceIm <[email protected]>
All green in build 14 (
|
From #8588 and the changes proposed in the reviews of such PR (including aacebedo#1)
closes #8588