[virt-tools-list] [PATCH 2/2] configure: cleanup {GLIB2, GTK}_CFLAGS

[Pavel Grunt]

Hi,

imo with the change the line is too long

About the AC_SUBST calls - iirc they are defined automatically when
pkg-config is >= 0.24 (not the case of rhel6).
Or is it about something else (why just _LIBS and not _CFLAGS) ?

Thanks,
Pavel

On Tue, 2016-03-15 at 11:48 -0300, Eduardo Lima (Etrunko) wrote:
> Also, remove unecessary AC_SUBST calls
> 
> Signed-off-by: Eduardo Lima (Etrunko) <etrunko at redhat.com>
> ---
>  configure.ac | 8 ++------
>  1 file changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 6d8475b..ee9dc73 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -102,10 +102,8 @@ GLIB_MKENUMS=`$PKG_CONFIG --
> variable=glib_mkenums glib-2.0`
>  AC_SUBST(GLIB_MKENUMS)
>  
>  PKG_CHECK_MODULES(GLIB2, glib-2.0 >= $GLIB2_REQUIRED gio-2.0
> gthread-2.0 gmodule-export-2.0)
> -GLIB2_CFLAGS="$GLIB2_CFLAGS
> -DGLIB_VERSION_MIN_REQUIRED=$GLIB2_ENCODED_VERSION"
> -GLIB2_CFLAGS="$GLIB2_CFLAGS
> -DGLIB_VERSION_MAX_ALLOWED=$GLIB2_ENCODED_VERSION"
> +GLIB2_CFLAGS="$GLIB2_CFLAGS
> -DGLIB_VERSION_MIN_REQUIRED=$GLIB2_ENCODED_VERSION
> -DGLIB_VERSION_MAX_ALLOWED=$GLIB2_ENCODED_VERSION"
>  AC_SUBST(GLIB2_CFLAGS)
> -AC_SUBST(GLIB2_LIBS)
>  
>  AC_ARG_VAR([GLIB_COMPILE_RESOURCES],[the glib-compile-resources
> program])
>  AC_PATH_PROG([GLIB_COMPILE_RESOURCES],[glib-compile-resources],[])
> @@ -137,10 +135,8 @@ AC_CHECK_LIB([virt],
>  LIBS=$old_LIBS
>  
>  PKG_CHECK_MODULES(GTK, gtk+-3.0 >= $GTK_REQUIRED)
> -GTK_CFLAGS="$GTK_CFLAGS
> -DGDK_VERSION_MIN_REQUIRED=$GTK_ENCODED_VERSION"
> -GTK_CFLAGS="$GTK_CFLAGS
> -DGDK_VERSION_MAX_ALLOWED=$GTK_ENCODED_VERSION"
> +GTK_CFLAGS="$GTK_CFLAGS
> -DGDK_VERSION_MIN_REQUIRED=$GTK_ENCODED_VERSION
> -DGDK_VERSION_MAX_ALLOWED=$GTK_ENCODED_VERSION"
>  AC_SUBST(GTK_CFLAGS)
> -AC_SUBST(GTK_LIBS)
>  
>  AC_ARG_WITH([gtk-vnc],
>      AS_HELP_STRING([--without-gtk-vnc], [Ignore presence of gtk-vnc
> and disable it]))