[ovs-dev] [ovn-controller-vtep V5 02/12] ovn-nbctl: Move ovn-nbctl to utilities directory.

Alex Wang alexw at nicira.com
Fri Aug 7 23:48:58 UTC 2015


On Fri, Aug 7, 2015 at 10:23 AM, Russell Bryant <rbryant at redhat.com> wrote:

> On 08/07/2015 01:18 PM, Russell Bryant wrote:
> > On 08/07/2015 03:46 AM, Alex Wang wrote:
> >> Signed-off-by: Alex Wang <alexw at nicira.com>
> >> ---
> >> V5:
> >> - new patch.
> >> ---
> >>  ovn/.gitignore                      |    2 --
> >>  ovn/automake.mk                     |   11 +++--------
> >>  ovn/utilities/.gitignore            |    3 +++
> >>  ovn/utilities/automake.mk           |   14 +++++++++++---
> >>  ovn/{ => utilities}/ovn-nbctl.8.xml |    0
> >>  ovn/{ => utilities}/ovn-nbctl.c     |    0
> >>  6 files changed, 17 insertions(+), 13 deletions(-)
> >>  rename ovn/{ => utilities}/ovn-nbctl.8.xml (100%)
> >>  rename ovn/{ => utilities}/ovn-nbctl.c (100%)
> >
> >> diff --git a/ovn/utilities/.gitignore b/ovn/utilities/.gitignore
> >> index 1ddc63e..714aaab 100644
> >> --- a/ovn/utilities/.gitignore
> >> +++ b/ovn/utilities/.gitignore
> >> @@ -1 +1,4 @@
> >>  /ovn-ctl.8
> >> +/ovn-nbctl
> >> +/ovn-nbctl.8
> >> +
> >
> > git complains that this patch introduces a whitespace error.  It's this
> > new blank line at the end of .gitignore.
> >
> > Otherwise:
> >
> > Acked-by: Russell Bryant <rbryant at redhat.com>
> >
>
> One more thing ... this can be fixed later if needed, but ovn/utilities/
> will need to be added to PATH in tutorial/ovs-sandbox.  After this
> patch, ovn-nbctl (and later ovn-sbctl) aren't in $PATH when running the
> sandbox.
>
>
Thx for pointing this out~!  Totally unaware,

Will also add it to utilities/ovs-sim.in



> --
> Russell Bryant
>



More information about the dev mailing list