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. -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev