Re: [ovs-dev] [PATCH] utilities/ovs-docker: Add support for specifying vlan tag on add-port cal

2016-06-20 Thread Guru Shetty
On 14 June 2016 at 17:02, Brian Turek wrote: > I have no idea why my patch wouldn't be working, `git diff -C -M` is > making the diff for me (if the below doesn't work again, I'll have to > investigate). > From this email, if you had to apply the patch on a clean

Re: [ovs-dev] [PATCH] utilities/ovs-docker: Add support for specifying vlan tag on add-port cal

2016-06-14 Thread Brian Turek
I have no idea why my patch wouldn't be working, `git diff -C -M` is making the diff for me (if the below doesn't work again, I'll have to investigate). Regardless, I really would like to have the tag option be a part of the initial add-port call instead of an additional set-port call. If

Re: [ovs-dev] [PATCH] utilities/ovs-docker: Add support for specifying vlan tag on add-port cal

2016-06-13 Thread Guru Shetty
On 11 June 2016 at 05:28, Brian Turek wrote: > Signed-off-by: Brian Turek > Thank you for your contribution. How did you send this patch? I couldn't apply it on my local tree using 'git am'. Sending it via 'git send-email' usually gets it right.

[ovs-dev] [PATCH] utilities/ovs-docker: Add support for specifying vlan tag on add-port cal

2016-06-11 Thread Brian Turek
Signed-off-by: Brian Turek diff --git a/utilities/ovs-docker b/utilities/ovs-docker index 43cea54..98892b6 100755 --- a/utilities/ovs-docker +++ b/utilities/ovs-docker @@ -90,6 +90,10 @@ add_port () { MTU=`expr X"$1" : 'X[^=]*=\(.*\)'`