[netcf-devel] [PATCH 3/3] (take two) Add a new API function ncf_if_xml_state().
David Lutterkort
lutter at redhat.com
Sun Sep 27 04:25:11 UTC 2009
On Sat, 2009-09-26 at 22:48 -0400, Laine Stump wrote:
> On 09/26/2009 08:28 PM, David Lutterkort wrote:
> > On Fri, 2009-09-25 at 09:36 -0400, Laine Stump wrote:
> >
> >> (I *think* I got all your suggestions in here, including rebase.
> >>
> > ACK. This looks good; I made some minor formatting changes and changed
> > the commit message + pushed.
> >
> > One minor nit: for interfaces with static IP's, you get something like
> > this:
> >
> > <?xml version="1.0"?>
> > <interface type="bridge" name="br0">
> > <start mode="onboot"/>
> > <protocol family="ipv4">
> > <ip address="172.31.0.9" prefix="24"/>
> > <route gateway="172.31.0.1"/>
> > <ip address="172.31.0.9" prefix="24" source="device"/>
> >
>
>
> Shoot! That wasn't intended, and it either wasn't working that way at
> some point, or I didn't notice the extra node when looking at the output
> (but it's there plain as day now). Do you want a patch to eliminate the
> duplicate? (It almost sounds like you prefer it this way.)
Yeah, a cleanup patch would be good.
David
More information about the netcf-devel
mailing list