GNOME Bugzilla – Bug 784975
Port to meson build system
Last modified: 2017-07-17 14:55:48 UTC
Created attachment 355670 [details] [review] Port to meson build system Complete port to meson build system.
I have also created a branch[0] to ease its testing. Any suggestion is very appreciated. [0] https://git.gnome.org/browse/gnome-disk-utility/log/?h=wip/inigomartinez/meson
Hello, wow, that's neat, so we can get meson support this cycle, that you so much! I can't tell how this should be tested, in a quick try all was fine - but the idea would be to ship both build systems for some time? If that's the case there is not much against putting it on master. Maybe someone experienced with meson can also acknowledge the patch. How does the release process differ? Should I stick to autotools in order to have the ./configure in the tar archive or is it ok ship the result of ninja dist? Regards, Kai
I don't know what is exactly the plan. I think that, at least, to have meson available as build system. There are some applications (totem is the one I worked on) that have removed autotools to avoid maintaining two build systems. Regarding release process, I have never done any, so I don't know. I can say that, from meson 0.41, dist command is available (ninja build dist), which may help on the release process.
Of course I am tempted to drop it but since Debian stretch only has an older meson, this excludes from contributing via simple clone and build with system libraries/tooling. At least until we figure out how the flatpak process could work in combinaiton with a system UDisks (the UDisks client library and D-Bus service versions do not match).
Created attachment 355700 [details] [review] Remove autotools As a follow up on the meson build path, this patch removes autotool as build system, just in case it is considered.
Created attachment 355703 [details] [review] Port to meson build system Minor improvement regarding meson build port patch, which changes the way scalable images are installed. Now the two SVG files are listed, and their existence checked, along their relative path instead of building their path when iterating through them.
Thanks a lot for working on this, the switch with droping autotools is now done.