adding gtkmm libraries to the next releases about gvsbuild HOT 6 CLOSED

Ok cool, would you be interested in making a PR to add Gtkmm to the GTK4 pipeline?

from gvsbuild.

Hi @alexv-ds, thanks for the suggestion! Adding Gtkmm would make sense to me because it is part of the core GTK libraries. However, we would need to add building of Gtkmm as part of our CI/CD pipeline which would slow down the builds. It would also increase the download size of the distributed zip. However, those are probably ok tradeoffs.

Can I ask what you are using Gtkmm for on Windows?

from gvsbuild.

Can I ask what you are using Gtkmm for on Windows?

I want to make another email client😅

Currently development is on Linux, but adding gtkmm will reduce the windows headaches🙂

from gvsbuild.

This is kind of a slippery slope... let's see the timing with Alex's patch and decide

Maybe we should revise a bit our CI strategy so that:

1. We do "nightly" builds with a fuller target
2. We run a smaller target for normal PRs
3. (optional but it be cool) If the PR contains a [ci target: foo] line we would build that target

The last point would also help with testing PR for libs that are not part of the default target

from gvsbuild.

Hi @pbor, I agree that we should be careful adding too much to the build and to the binaries. It looks like this didn't really change our build time, and it looks like the binaries are about 30mb larger. Maybe we should clearly define what is and what is not part of our core part of GTK4 that we want to provide. For example, libraries like libsoup, glib-networking, and graphene aren't so clear to me - they are definitely GTK related.

I also agree that using something like https://github.com/tj-actions/changed-files to control the pipeline would be really nice to build a much more targeted build for what has changed. Probably for tools updates we'll need to build a wider set of libraries, but we probably don't need to build all of GTK if an unrelated library updates.

from gvsbuild.

Closed by #1348. Thanks @alexv-ds!

from gvsbuild.