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

Reply via email to