1 From 5a406b06792e26a83c7346b3c2443c0bd8d4cdb2 Mon Sep 17 00:00:00 2001
2 From: Eli Schwartz <eschwartz@archlinux.org>
3 Date: Mon, 8 Nov 2021 18:22:47 -0500
4 Subject: [PATCH] migrate from custom itstool to builtin msgfmt for creating
7 gettext upstream has supported this for a very long time (since 0.19.7
8 via commit b3c2a5a242c36fbbaa0c5b17f975d6c638598a23, released in 2015),
9 and itstool is (mostly) a legacy of the time before gettext had proper
10 support for these sorts of use cases.
12 This is similar to the state of intltool, which is described at
13 https://wiki.gnome.org/MigratingFromIntltoolToGettext
15 During the port from autotools to meson, the legacy use of itstool was
16 faithfully translated to meson in the only way possible: by jumping
17 through hoops to run ninja inside ninja in order to generate the .mo
18 files for itstool, because meson's i18n module used a flawed design and
19 there was no "real" target to create those files, only a .PHONY
20 run_target which other rules cannot depend on.
22 Although meson 0.60.0 added support for real targets for the built .mo
23 files, this changed the rules for output filenames, breaking the script.
25 But msgfmt does not care, and anyways comes with builtin meson functions
26 for convenient use with XML files. So let's take this opportunity to
27 drop legacy dependencies and use the modern, builtin tooling, which
28 fixes this bug as a side effect.
34 data/freedesktop_generate.sh | 12 ------------
35 data/meson.build | 16 +++++-----------
37 5 files changed, 6 insertions(+), 27 deletions(-)
38 delete mode 100755 data/freedesktop_generate.sh
40 diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
41 index 985dfe9..345e1ce 100644
44 @@ -9,7 +9,6 @@ variables:
52 @@ -34,4 +33,3 @@ build:meson:
54 - _build/meson-logs/*.txt
55 - _build/meson-dist/*.*
57 diff --git a/README.md b/README.md
58 index c23d9d5..b2fb0ac 100644
61 @@ -19,7 +19,7 @@ $ ninja -v -C _build install
64 It requires the glib development package to be installed as well as
67 This database is translated at Transifex.
70 diff --git a/data/freedesktop_generate.sh b/data/freedesktop_generate.sh
71 deleted file mode 100755
72 index 62ccffa..0000000
73 --- a/data/freedesktop_generate.sh
81 -${NINJA:-ninja} -C "${build_root}" shared-mime-info-gmo
84 - --its "${src_root}/data/its/shared-mime-info.its" \
85 - --join "${src_root}/data/freedesktop.org.xml.in" \
86 - -o "${build_root}/data/freedesktop.org.xml" \
87 - "${build_root}/po/"*".gmo"
88 diff --git a/data/meson.build b/data/meson.build
89 index 24361c9..09ed7a9 100644
90 --- a/data/meson.build
91 +++ b/data/meson.build
94 install_man('update-mime-database.1')
96 -freedesktop_org_xml = custom_target('freedesktop.org.xml',
98 - 'freedesktop.org.xml.in',
99 - 'its/shared-mime-info.its',
100 - 'its/shared-mime-info.loc',
102 +freedesktop_org_xml = i18n.merge_file(
103 + input: 'freedesktop.org.xml.in',
104 output: 'freedesktop.org.xml',
106 - find_program('freedesktop_generate.sh'),
107 - meson.source_root(),
114 install_dir: get_option('datadir') / 'mime' / 'packages',
116 diff --git a/meson.build b/meson.build
117 index 0d08c8a..60f17ae 100644
120 @@ -20,7 +20,6 @@ config.set_quoted('VERSION', meson.project_version())
121 ###############################################################################
124 -itstool = find_program('itstool')
125 xmllint = find_program('xmllint')
126 xmlto = find_program('xmlto', required: false)