[libvirt] [PATCH 1/5] build: clean up CPPFLAGS/INCLUDES usage
Jiri Denemark
jdenemar at redhat.com
Fri Feb 3 08:55:01 UTC 2012
On Thu, Feb 02, 2012 at 17:33:20 -0700, Eric Blake wrote:
> Our syntax checker missed all-lower-case variables (this will
> be fixed by the next .gnulib update). Additionally, anywhere
> that we mix in-tree files with generated files, automake recommends
> listing builddir prior to srcdir for VPATH builds.
>
> * src/Makefile.am (*_la_CFLAGS): Favor $(top_srcdir).
> (INCLUDES): Likewise, and follow automake recommendations on
> builddir before srcdir.
> * python/Makefile.am (INCLUDES): Swap directory order.
> * tests/Makefile.am (INCLUDES): Likewise.
> * tools/Makefile.am (INCLUDES): Likewise.
> * daemon/Makefile.am (INCLUDES): Likewise.
> (libvirtd.init, libvirtd.service): Favor $().
> * examples/hellolibvirt/Makefile.am (hellolibvirt_LDADD):
> Likewise.
> * examples/openauth/Makefile.am (openauth_LDADD): Likewise.
> * examples/dominfo/Makefile.am (INCLUDES): Drop dead include.
> * examples/domsuspend/Makefile.am (INCLUDES): Likewise.
> ---
> daemon/Makefile.am | 20 +++++-----
> examples/dominfo/Makefile.am | 4 +-
> examples/domsuspend/Makefile.am | 4 +-
> examples/hellolibvirt/Makefile.am | 4 +-
> examples/openauth/Makefile.am | 4 +-
> python/Makefile.am | 4 +-
> src/Makefile.am | 70 ++++++++++++++++++------------------
> tests/Makefile.am | 9 ++---
> tools/Makefile.am | 12 +++---
> 9 files changed, 65 insertions(+), 66 deletions(-)
ACK
Jirka
More information about the libvir-list
mailing list