-
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
Openscenegraph/3.6.5 opengl profile #15366
Openscenegraph/3.6.5 opengl profile #15366
Conversation
Signed-off-by: Uilian Ries <uilianries@gmail.com>
…file Update tools
This comment has been minimized.
This comment has been minimized.
I detected other pull requests that are modifying openscenegraph/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.
…g libtiff/4.3.0 dependency which itself depends on libjpeg/9e (and not libjpeg/9d)
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit f8058d8openscenegraph/3.6.5
|
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.
Works with my dev environment.
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.
Tested successfully on my environment.
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.
Works for me thank you ! 🐱👤
You should consider making this recipe ready for v2 :) |
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 could be improved for Conan v2, but so far, it's good enough for a new option PR.
9621f19
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.
This recipe needs a lot of improvements :) you should consider following it up if you want to keep it working
This comment has been minimized.
This comment has been minimized.
I dont know why bzip was missing but hopefully I fixed the other problems |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ✔️All green in build 9 (
Conan v2 pipeline (informative, not required for merge) ❌
The v2 pipeline failed. Please, review the errors and note this will be required for pull requests to be merged in the near future. See details:Failure in build 11 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
Hooks produced the following warnings for commit 149bb6bopenscenegraph/3.6.5
|
It would be convenient if the "opengl_profile" attribute was exposed in the conan recipe. It currently works like a charm on my env.
I didn't have the time to take care of the merge of this PR last year, that's why I'm reopening this PR.