-
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
[gtk]: Enable gtk3 on windows and use shared glib when shared #11308
Conversation
I detected other pull requests that are modifying gtk/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.
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.
Hooks produced the following warnings for commit f4a621dgtk/4.7.0
gtk/4.3.2
gtk/4.4.0
gtk/4.6.2
gtk/4.2.1
|
the meson build script for gtk3 has: - cairo_dep = dependency('cairo', version: cairo_req, required: cc.get_id() != 'msvc') that is cairo is not required if msvc, but this is because they use another search criteria right after for windows - if not cairo_dep.found() and cc.get_id() == 'msvc' - if cc.has_header('cairo.h') and cc.has_header('cairo-win32.h') - cairo_dep = cc.find_library('cairo', required: false) - endif - endif - - if not cairo_dep.found() - cairo_dep = dependency('cairo', version: cairo_req, - fallback : ['cairo', 'libcairo_dep']) - endif and eventually cairo has to be found for gtk to be built
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.
Failure in build 31 (
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. |
Specify library name and version: gtk
The major blocker for building gtk 3 on windows was LNK1170 error ; that is the linking command for gtk 3 was so big the msvc linker can't handle it even with short_paths on.
This patch fixes this by editing the
ninja.build
file and removing duplicates in the link arguments.It's worth noting that for static gtk to work, it has to be built from scratch on the host device, since the location of the resources will be hard coded, and as such I propose to make gtk shared by default.