[libvirt] [PATCH] build: simplify sanlock distribution

[Eric Blake]

On 06/29/2011 03:38 AM, Daniel P. Berrange wrote:
>> -
>> -$(builddir)/locking/%-sanlock.conf: $(srcdir)/locking/sanlock.conf
>> -	$(AM_V_GEN)mkdir locking ; \
>> -	cp $< $@
> 
> How does  'qemu-sanlock.conf' get created now ? This rule is what actually
> tells make how to create 'qemu-sanlock.conf' from 'sanlock.conf'

Serves me right for coding too late at night.

I guess you have a point - you want to ship sanlock.conf as a template,
as well as install multiple <HV>-sanlock.conf files, one for each <HV>
that supports sanlock (right now, just qemu), while still storing only
one file in libvirt.git.  I missed the '%-' in the rule in my first read
of it.

In which case, my patch is wrong.  But the existing code is also wrong -
the EXTRA_DIST still needs to be moved outside of HAVE_SANLOCK, and the
'mkdir' needs to instead be '$(MKDIR_P)' for parallel builds.

v2 coming up.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110629/6472766c/attachment-0001.sig>