<br><tt><font size=2>"Daniel P. Berrange" <berrange@redhat.com>
wrote on 03/17/2010 07:52:53 AM:<br>
<br>
</font></tt>
<br><tt><font size=2>> <br>
> On Thu, Mar 11, 2010 at 08:09:13AM -0500, Stefan Berger wrote:<br>
> > This patch adds virsh support for the five new CLI commands to
manage<br>
> > network filters.<br>
> > <br>
> > Signed-off-by: Stefan Berger <stefanb@us.ibm.com><br>
> > <br>
> <br>
> > ---<br>
> >  tools/virsh.c |  349 ++++++++++++++++++++++++++++++++++++++++++++<br>
> ++++++++++++++<br>
> >  1 file changed, 349 insertions(+)<br>
> > <br>
> > Index: libvirt-acl/tools/virsh.c<br>
> > ===================================================================<br>
> > --- libvirt-acl.orig/tools/virsh.c<br>
> > +++ libvirt-acl/tools/virsh.c<br>
> > @@ -252,6 +252,14 @@ static virNetworkPtr vshCommandOptNetwor<br>
> >      vshCommandOptNetworkBy(_ctl, _cmd, _name,
                     \<br>
> >                  
          VSH_BYUUID|VSH_BYNAME)<br>
> >  <br>
> > +static virNWFilterPtr vshCommandOptNWFilterBy(vshControl *ctl,
<br>
> const vshCmd *cmd,<br>
> > +                  
                     
         char **name, int flag);<br>
> > +<br>
> > +/* default is lookup by Name and UUID */<br>
> > +#define vshCommandOptNWFilter(_ctl, _cmd, _name)    
               \<br>
> > +    vshCommandOptNWFilterBy(_ctl, _cmd, _name,  
                   \<br>
> > +                  
         VSH_BYUUID|VSH_BYNAME)<br>
> > +<br>
> >  static virInterfacePtr vshCommandOptInterfaceBy(vshControl
*ctl, <br>
> const vshCmd *cmd,<br>
> >                  
                     
         char **name, int flag);<br>
> >  <br>
> > @@ -3808,6 +3816,300 @@ cmdInterfaceDestroy(vshControl *ctl,
con<br>
> >      return ret;<br>
> >  }<br>
> >  <br>
> > +<br>
> > +/*<br>
> > + * "nwfilter-define" command<br>
> > + */<br>
> > +static const vshCmdInfo info_nwfilter_define[] = {<br>
> > +    {"help", gettext_noop("define or
update a network filter from<br>
> an XML file")},<br>
> > +    {"desc", gettext_noop("Define a
new network filter or update <br>
> an existing one.")},<br>
> > +    {NULL, NULL}<br>
> > +};<br>
> <br>
> Since you wrote this, we've decide to replace 'gettext_noop' with
'N_' <br>
> in all virsh code. 'make syntax-check' should warn about this now,
i <br>
> believe</font></tt>
<br>
<br><tt><font size=2>Thanks for pointing this out -- fixed.</font></tt>
<br><tt><font size=2>Unfortunately 'make syntax-check' does not report
the warning. Also, what package provides the 'cppi' tool that is being
used during the syntax check?</font></tt>
<br>
<br><tt><font size=2>Thanks and regards,</font></tt>
<br><tt><font size=2>    Stefan</font></tt>
<br>
<br><tt><font size=2><br>
> <br>
> Regards,<br>
> Daniel<br>
> -- <br>
> |: Red Hat, Engineering, London    -o-   </font></tt><a href=http://people.redhat.com/berrange/:|><tt><font size=2>http://people.redhat.com/berrange/:|</font></tt></a><tt><font size=2><br>
> |: </font></tt><a href=http://libvirt.org/><tt><font size=2>http://libvirt.org</font></tt></a><tt><font size=2>
-o- </font></tt><a href="http://virt-manager.org/"><tt><font size=2>http://virt-manager.org</font></tt></a><tt><font size=2>
-o- </font></tt><a href=http://deltacloud.org:|/><tt><font size=2>http://deltacloud.org:|</font></tt></a><tt><font size=2><br>
> |: </font></tt><a href=http://autobuild.org/><tt><font size=2>http://autobuild.org</font></tt></a><tt><font size=2>
       -o-         </font></tt><a href=http://search.cpan.org/~danberr/:|><tt><font size=2>http://search.cpan.org/~danberr/:|</font></tt></a><tt><font size=2><br>
> |: GnuPG: 7D3B9505  -o-   F3C9 553F A1DA 4AC2 5648 23C1
B3DF F742 7D3B 9505 :|<br>
</font></tt>