Thanks a lot!
/Jan

> -----Original Message-----
> From: Ben Pfaff [mailto:[email protected]]
> Sent: Wednesday, 08 November, 2017 21:43
> To: Jan Scheurich <[email protected]>
> Cc: '[email protected]' <[email protected]>
> Subject: Re: [PATCH v2 1/2] NSH: Minor bugfixes
> 
> The attachments worked.  I applied them to master and branch-2.8.  I
> added a NEWS entry:
> 
>    - NSH implementation now conforms to latest draft
>      (draft-ietf-sfc-nsh-28).
> 
> Thank you!
> 
> On Wed, Nov 08, 2017 at 08:48:53AM +0000, Jan Scheurich wrote:
> > Hi Ben,
> >
> > I think the first patch does not apply because somehow the email process 
> > has converted the leading tab to 7 spaces in line 24 of the
> patch.
> >
> > Unfortunately I can't use git send-email in our company network due to 
> > firewalls restrictions. Could you try to use the attached patch
> files instead?
> >
> > BR, Jan
> >
> > > -----Original Message-----
> > > From: Ben Pfaff [mailto:[email protected]]
> > > Sent: Wednesday, 08 November, 2017 05:27
> > > To: Jan Scheurich <[email protected]>
> > > Cc: '[email protected]' <[email protected]>; Yang, Yi Y 
> > > ([email protected]) <[email protected]>; Jiri Benc
> > > ([email protected]) <[email protected]>
> > > Subject: Re: [PATCH v2 1/2] NSH: Minor bugfixes
> > >
> > > On Tue, Nov 07, 2017 at 03:34:06PM +0000, Jan Scheurich wrote:
> > > > - Fix 2 incorrect length checks
> > > > - Remove unnecessary limit of MD length to 16 bytes
> > > > - Remove incorrect comments stating MD2 was not supported
> > > > - Pad metadata in encap_nsh with zeroes if not multiple of 4 bytes
> > > >
> > > > Signed-off-by: Jan Scheurich <[email protected]>
> > >
> > > There's something odd about this patch and it does not apply:
> > >
> > >     Using index info to reconstruct a base tree...
> > >     error: patch failed: 
> > > datapath/linux/compat/include/linux/openvswitch.h:793
> > >     error: datapath/linux/compat/include/linux/openvswitch.h: patch does 
> > > not apply
> > >     error: Did you hand edit your patch?
> > >     It does not apply to blobs recorded in its index.
> > >
> > > I've never seen that particular error before.

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to