[libvirt] [PATCH] (updated) additional parameters needed for dnsmasq

[Eric Blake]

On 08/22/2012 11:39 AM, Eric Blake wrote:
> On 08/22/2012 10:59 AM, Gene Czarcinski wrote:
>> As I said in a previous message, dnsmasq is forwarding a number of
>> queries upstream that should not be done.  There still remains an MX
>> query for a plain name with no domain specified that will be forwarded
>> is dnsmasq has --domain=xxx  --local=/xxx/ specified. This does not
>> happen with no domain name and --local=// ... not a libvirt problem.
>>
> 
> ACK and pushed with the above tweak, and with adding you to AUTHORS (let
> us know if you prefer any other spelling or email address; the file is
> in UTF-8).

Oh, and now that I've already pushed, I have a high-level question: what
is the minimum version of 'dnsmasq' that supports the command-line
syntax that this patch introduces?

+--local=// --domain-needed --filterwin2k \

If older dnsmasq doesn't recognize --local=// or the new --domain-needed
or --filterwin2k options, then we either need to make this code
conditional based on probing 'dnsmasq --help' at startup, or else change
the spec file to require a larger minimum version of dnsmasq (we already
require 2.41 for other reasons).

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

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