On Mon, Jan 07, 2019 at 12:00:47PM +0400, marcandre.lur...@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lur...@redhat.com>
> 
> Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> ---
>  meson.build       | 32 +++++++++++++++-----------------
>  meson_options.txt |  3 +--
>  2 files changed, 16 insertions(+), 19 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index 18e330f..07e8227 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -186,26 +186,24 @@ endif
>  
>  # polkit
>  spice_gtk_has_polkit = false
> -if get_option('polkit')
> -  polkit_dep = dependency('polkit-gobject-1', version : '>= 0.96')# 
> ,required : false)
> -  if polkit_dep.found()
> -    spice_gtk_policy_dir = polkit_dep.get_pkgconfig_variable('policydir')
> -    foreach func : ['polkit_authority_get_sync', 
> 'polkit_authorization_result_get_dismissed']
> -      if compiler.has_function(func, dependencies : polkit_dep)
> -        spice_gtk_config_data.set('HAVE_@0@'.format(func.to_upper()), '1')
> -      endif
> -    endforeach
> -
> -    if not compiler.has_function('acl_get_file')
> -      acl_dep = compiler.find_library('acl')
> -      if not compiler.has_function('acl_get_file', dependencies : acl_dep)
> -        error('PolicyKit support requested, but some required packages are 
> not available')
> -      endif
> -      spice_acl_deps += acl_dep
> +d = dependency('polkit-gobject-1', version : '>= 0.96', required : 
> get_option('polkit'))
> +if d.found()
> +  spice_gtk_policy_dir = d.get_pkgconfig_variable('policydir')
> +  foreach func : ['polkit_authority_get_sync', 
> 'polkit_authorization_result_get_dismissed']
> +    if compiler.has_function(func, dependencies : d)
> +      spice_gtk_config_data.set('HAVE_@0@'.format(func.to_upper()), '1')
> +    endif
> +  endforeach
> +
> +  if not compiler.has_function('acl_get_file')
> +    acl_dep = compiler.find_library('acl')
> +    if not compiler.has_function('acl_get_file', dependencies : acl_dep)
> +      error('PolicyKit support requested, but some required packages are not 
> available')

With 'auto', we should just disable polkit support if this is missing.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to