On Fri, Aug 7, 2015 at 10:23 AM, Russell Bryant <rbry...@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 <al...@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 <rbry...@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
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
