[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH v5] Implement DNS SRV record into the bridge driver



On Mon, Jan 02, 2012 at 03:23:54PM +0100, Michal Novotny wrote:
> Hi,
> this is the fifth version of my SRV record for DNSMasq patch rebased
> for the current codebase to the bridge driver and libvirt XML file to
> include support for the SRV records in the DNS. The syntax is based on
> DNSMasq man page and tests for both xml2xml and xml2argv were added as
> well. There are some things written a better way in comparison with
> version 4, mainly there's no hack in tests/networkxml2argvtest.c and
> also the xPath context is changed to use a simpler query using the
> virXPathInt() function relative to the current node.
> 
> Also, the patch is also fixing the networkxml2argv test to pass both
> checks, i.e. both unit tests and also syntax check.
> 
> Please review,

  Yeah, it looks good to me, actually we should have pushed it before in
december but I forgot it. Since it's relatively contained, I'm pushing
it in spite of the freeze, we already delayed it over one release. Blame
me if it breaks something it was my fault I forgot the patch !

  ACK and pushed,

    thanks !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]