qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 3/8] configure: move gettext detection to meson.build


From: Paolo Bonzini
Subject: Re: [PATCH v2 3/8] configure: move gettext detection to meson.build
Date: Wed, 16 Dec 2020 16:35:24 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0

On 16/12/20 16:27, Alex Bennée wrote:

Paolo Bonzini <pbonzini@redhat.com> writes:

On 16/12/20 15:19, Alex Bennée wrote:

Alex Bennée <alex.bennee@linaro.org> writes:

This will allow meson to honour -Dauto_features=disabled later.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

ping Paolo, are you happy with this bit?

Yes, I'm so happy with this and the --with-default-features parts that I
was going to include it in my next pull request. :)

I'm putting together an testing one now (as there is a testing update as
well).

Great, I'll rebase when it hits qemu.git.

Paolo


Series

Acked-by: Paolo Bonzini <pbonzini@redhat.com>

Paolo

---
   configure         | 19 +++----------------
   meson_options.txt |  2 +-
   po/meson.build    |  2 +-
   3 files changed, 5 insertions(+), 18 deletions(-)

diff --git a/configure b/configure
index 8f2095a2db..46fdea88c7 100755
--- a/configure
+++ b/configure
@@ -448,7 +448,7 @@ libdaxctl=""
   meson=""
   ninja=""
   skip_meson=no
-gettext=""
+gettext="auto"
bogus_os="no"
   malloc_trim="auto"
@@ -1014,9 +1014,9 @@ for opt do
     ;;
     --enable-vnc) vnc="enabled"
     ;;
-  --disable-gettext) gettext="false"
+  --disable-gettext) gettext="disabled"
     ;;
-  --enable-gettext) gettext="true"
+  --enable-gettext) gettext="enabled"
     ;;
     --oss-lib=*) oss_lib="$optarg"
     ;;
@@ -2839,19 +2839,6 @@ if test "$xen_pci_passthrough" != "disabled"; then
     fi
   fi
-##########################################
-# gettext probe
-if test "$gettext" != "false" ; then
-  if has xgettext; then
-    gettext=true
-  else
-    if test "$gettext" = "true" ; then
-      feature_not_found "gettext" "Install xgettext binary"
-    fi
-    gettext=false
-  fi
-fi
-
   ##########################################
   # X11 probe
   if $pkg_config --exists "x11"; then
diff --git a/meson_options.txt b/meson_options.txt
index f6f64785fe..191e848600 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -9,7 +9,7 @@ option('sphinx_build', type : 'string', value : '',
option('docs', type : 'feature', value : 'auto',
          description: 'Documentations build support')
-option('gettext', type : 'boolean', value : true,
+option('gettext', type : 'feature', value : 'auto',
          description: 'Localization of the GTK+ user interface')
   option('install_blobs', type : 'boolean', value : true,
          description: 'install provided firmware blobs')
diff --git a/po/meson.build b/po/meson.build
index 1387fd979a..a863f0575f 100644
--- a/po/meson.build
+++ b/po/meson.build
@@ -1,6 +1,6 @@
   i18n = import('i18n')
-if get_option('gettext')
+if find_program('xgettext', required: get_option('gettext')).found()
     i18n.gettext(meson.project_name(),
                  args: '--msgid-bugs-address=qemu-devel@nongnu.org',
                  preset: 'glib')








reply via email to

[Prev in Thread] Current Thread [Next in Thread]