[Libvir] [PATCH] Fix substitution of QEMUD_PID_FILE
Richard W.M. Jones
rjones at redhat.com
Tue Apr 3 11:47:36 UTC 2007
Mark McLoughlin wrote:
> On Tue, 2007-04-03 at 11:40 +0100, Richard W.M. Jones wrote:
>
>> Use AC_SUBST instead of AC_DEFINE because we want autoconf to
>> substitute
>> the value into the Makefile, not into config.h. Then add
>> @QEMUD_PID_FILE@ to CFLAGS.
>
> Sounds sane ...
>
> Bear in mind, though, that ${prefix}/var is the default for
> $localstatedir ... but all Fedora packages would be built with
> --localstatedir=/var, just like --sysconfdir=/etc
>
> I don't think you see the problem if you pass a value for
> --localstatedir
>
>> + -DQEMUD_PID_FILE="\"@QEMUD_PID_FILE@\""
>
> Use $(QEMU_PID_FILE)
OK, that works as well.
Are we good to commit this patch and the preceeding one (adding help to
libvirt qemu daemon)?
Rich.
--
Emerging Technologies, Red Hat http://et.redhat.com/~rjones/
64 Baker Street, London, W1U 7DF Mobile: +44 7866 314 421
Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod
Street, Windsor, Berkshire, SL4 1TE, United Kingdom.
Registered in England and Wales under Company Registration No. 3798903
Directors: Michael Cunningham (USA), Charlie Peters (USA) and David
Owens (Ireland)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3237 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20070403/92d1e45c/attachment-0001.bin>
More information about the libvir-list
mailing list