On Tue, Aug 16, 2011 at 09:15:12AM -0400, Laine Stump wrote:
On 08/14/2011 10:49 AM, Zdenek Styblik wrote:
>On 08/14/11 16:17, Zdenek Styblik wrote:
>>Hi,
>>
>>I've noticed libvirt is persisting to have dnsmasq present and use it.
>>Version of libvirt in question is 0.9.4.
>>
>>There is no dnsmasq present on the Host during libvirt compilation, yet
>>it seems to me ./configure wrongfully finds one:
>>
>>~~~ SNIP ~~~
>>configure:46087: checking for dnsmasq
>>configure:46119: result: dnsmasq
>>~~~ SNIP ~~~
>>
>>shouldn't there be "result: no"? As far as I remember - no dnsmasq
>>present during compilation, no usage of it later. However I might have
>>missed something - change in behavior, requirements etc. This is
>>possible. Still, if required "component" is missing, I say compilation
>>resp. configure should fail.
Well, libvirt.spec has had Require: dnsmasq for at least a couple
years, so any install of a libvirt rpm should fail when dnsmasq
isn't present. I don't know for certain how long the BuildRequires:
dnsmasq has been there, but certainly for at least a year (the last
time the line was modified), so any rpm builds should also fail.
But of course if you're running ./autogen.sh and then make, that
doesn't involve the specfile.
dnsmasq really is an integral part of the network driver; I don't
know that it makes any sense to "fix" things so it can be built
without dnsmasq. It's probably a good idea to make the failure
complete though.
We already have an option to disable the virtual network driver. I don't
think we want to have a separate option for dnsmasq, since that is an
integral part of hte network driver IMHO
Daniel
--
|:
http://berrange.com -o-
http://www.flickr.com/photos/dberrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|:
http://entangle-photo.org -o-
http://live.gnome.org/gtk-vnc :|