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@xxxxxxxxxx +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list