feat! (major-refactor): break gtk into multiple build-targets#4339
feat! (major-refactor): break gtk into multiple build-targets#4339
Conversation
|
@3v1n0 from earlier discussion: do you think we should split gtk package into:
then we move |
|
Diff is too big for github to handle it, however on: # Create assets directory and run the render script directly
gtk2_assets_target = custom_target(
'@0@-gtk2-assets'.format(theme_name),
input: [gtk2_svg_to_use, gtk2_assets_txt],
output: '@0@-gtk2-assets.stamp'.format(theme_name),
command: [
'bash', gtk2_render_script_path,
'@INPUT0@', # SVG file
'@INPUT1@', # assets.txt
'@PRIVATE_DIR@/assets', # output directory
'--clean'
],
build_by_default: true,
)I think we should do something similar that we do for icons too, so that both SVGs and PNGs are generated only by CI and not by developers unless they don't run a custom target that is only exposed for that reason, but not done while doing the simple |
|
Regarding commit ecd796c I'm unsure if we want to go there, that would imply also quite a bit of synchronization distro-side, and prepare migration scripts. While this needs design review (/cc @juanruitina) I feel that it would be better to leave it post 26.04, to avoid having to deal with such migration in the LTS cycle (/cc @aleasto). |
debian/control
Outdated
| inkscape, | ||
| optipng, |
There was a problem hiding this comment.
No please, if we need to generate something, let's do it in CI, to avoid having the builders to do all the job
There was a problem hiding this comment.
sure, will remove these
gtk/src/gtk-2.0/meson.build
Outdated
| # Track previous flavour's assets target to enforce sequential builds | ||
| gtk2_prev_assets_target = [] |
There was a problem hiding this comment.
Why this?
If we need to re-use stuff for the dark side I'd prefer to have explicit dependencies.
Not great, but we can use also set_variable and get_variable to access to specific variables that are defined programmatically
There was a problem hiding this comment.
I think I'll drop this logic in favour of icons render logic that we use with ci tooling
56870af to
e5e13b0
Compare
|
Hi @3v1n0 I force-pushed on top of my previous approach and here is the tldr;:
Questions:
|
gtk
upstream: use upstream colors for $selected_bg_colorwill be done post 26.04 release