Re: [libvirt] libvirt build failure w/GNU make and automake.git (automake regression?)

[Eric Blake]

On 09/12/2012 09:01 AM, Jim Meyering wrote:
> When I run ./autogen.sh && make, I see this:
> (this arose because I had the latest automake.git/master tools --
>  commit c1b83e1af60b866cf5cdeebf77d0275019bad8b2 from today --
>  early in my path)
> 

>     Making all in tests
>     make[2]: Entering directory `/h/j/w/co/libvirt/tests'
>     Makefile:4355: *** Malformed target-specific variable definition.  Stop.

> 
> The trouble is that "undefine" is an operator in GNU make.

> The most pragmatic work-around is to rename the "undefine" test script.

Indeed - while the upstream debate continues on whether 'make',
'automake', or both should be patched to allow 'undefine', downstream in
libvirt, I am pushing this trivial patch:

From a20f06d9d9b0353d7fb7a8e11a631253d5961b96 Mon Sep 17 00:00:00 2001
From: Eric Blake <address@hidden>
Date: Wed, 12 Sep 2012 11:25:51 -0600
Subject: [PATCH] build: avoid confusing make with raw name 'undefine'

Make has a builtin operator 'undefine', and coupled with latest
automake.git, this test name ended up confusing make into thinking
the file name was meant to be used as the make operator.  Renaming
the file avoids the confusion.

* tests/undefine: Rename...
* tests/virsh-undefine: ...to this.
* tests/Makefile.am (test_scripts): Use new name.
Reported by Jim Meyering.
---
 tests/Makefile.am                  | 10 ++++++----
 tests/{undefine => virsh-undefine} |  0
 2 files changed, 6 insertions(+), 4 deletions(-)
 rename tests/{undefine => virsh-undefine} (100%)

diff --git a/tests/Makefile.am b/tests/Makefile.am
index bec89e2..c5cecaa 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -182,12 +182,13 @@ test_scripts +=                           \
        read-bufsiz                     \
        read-non-seekable               \
        start                           \
-       undefine                        \
        vcpupin                         \
        virsh-all                       \
        virsh-optparse                  \
        virsh-schedinfo                 \
-       virsh-synopsis
+       virsh-synopsis                  \
+       virsh-undefine                  \
+       $(NULL)

 test_programs +=                       \
        eventtest                       \
@@ -203,12 +204,13 @@ EXTRA_DIST +=                             \
        read-bufsiz                     \
        read-non-seekable               \
        start                           \
-       undefine                        \
        vcpupin                         \
        virsh-all                       \
        virsh-optparse                  \
        virsh-schedinfo                 \
-       virsh-synopsis
+       virsh-synopsis                  \
+       virsh-undefine                  \
+       $(NULL)
 endif

 if WITH_SECDRIVER_APPARMOR
diff --git a/tests/undefine b/tests/virsh-undefine
similarity index 100%
rename from tests/undefine
rename to tests/virsh-undefine
-- 
1.7.11.4



-- 
Eric Blake   address@hidden    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature