Re: [vdagent-win PATCH 6/6] Makefile.am: add INCLUDES to AM_CPPFLAGS

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

 



On Thu, Nov 7, 2013 at 11:02 PM, Uri Lublin <uril@xxxxxxxxxx> wrote:
> Fixes the following autoreconf (automake) warning:
> Makefile.am:6: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
>
> diff --git a/Makefile.am b/Makefile.am
> index da0b5a5..fc017d5 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -3,13 +3,13 @@ MAINTAINERCLEANFILES =
>
>  DIST_SUBDIRS = spice-protocol
>
> -INCLUDES =                             \
> +INC =                                  \
>         -I$(top_srcdir)/common          \
>         -I$(top_srcdir)/spice-protocol  \
>         $(NULL)
>

The renaming is worse :) I would just move that to AM_CPPFLAGS declaration.

>  AM_CXXFLAGS = -flto -fwhole-program
> -AM_CPPFLAGS = -DUNICODE -D_UNICODE -DOLDMSVCRT
> +AM_CPPFLAGS = -DUNICODE -D_UNICODE -DOLDMSVCRT $(INC)
>  AM_LDFLAGS = -static
>  if !ENABLE_DEBUG
>  AM_LDFLAGS += -s
> --
> 1.8.3.1
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel



-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]