[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: |
Alex Bennée |
Subject: |
Re: [PATCH v2 3/8] configure: move gettext detection to meson.build |
Date: |
Wed, 16 Dec 2020 14:19:59 +0000 |
User-agent: |
mu4e 1.5.7; emacs 28.0.50 |
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?
> ---
> 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')
--
Alex Bennée
[PATCH v2 4/8] configure: add --without-default-features, Alex Bennée, 2020/12/10
[PATCH v2 2/8] gitlab: include aarch64-softmmu and ppc64-softmmu cross-system-build, Alex Bennée, 2020/12/10
[PATCH v2 1/8] configure: include moxie-softmmu in deprecated_targets_list, Alex Bennée, 2020/12/10