-
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
gobject-introspection 1.66.1 #4254
gobject-introspection 1.66.1 #4254
Conversation
38bca5d
to
cf0d1d6
Compare
Some configurations of 'gobject-introspection/1.66.1' failed in build 1 (
|
Some configurations of 'gobject-introspection/1.66.1' failed in build 2 (
|
also, fix UB in test_package.c
All green in build 3 (
|
|
||
def config_options(self): | ||
if self.settings.os == "Windows": | ||
del self.options.fPIC |
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.
del self.options.fPIC | |
del self.options.fPIC | |
def validate(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.
I had very strange bugs on CCI when using validate(self) the last days:
eg #4181 (comment) or #4182 (comment)
so I'm not very inclined to use it until we understand what happened. Can we wait to introduce 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.
#4193 too
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.
Fair enough, validate
is experimental and even missing from the Conan changelog :) Hope we would refactor recipes to use validate
later.
Specify library name and version: gobject-introspection/1.66.1
try FreeBSD
conan-center hook activated.