On Fri, Jan 27, 2023 at 06:05:57PM +0100, Ilya Maximets wrote: > On 1/27/23 14:34, Simon Horman wrote: > > This series addresses an unnecessary and missing dependency on 'nc' in > > system tests. > > > > Patch 1/2: Remove unnecessary dependency on nc > > Patch 2/2: Add necessary dependency on nc > > > > As reported by David Marchand <[email protected]> > > Ref: https://mail.openvswitch.org/pipermail/ovs-dev/2023-January/401399.html > > > > Simon Horman (2): > > system-traffic: remove unnecessary dependency on nc > > system-offloads-traffic: skip tests if nc is not present > > I'd capitalize the first letter after the colon and add a > period at the end while applying.
Thanks, I did that while applying these to master. As per discussion elsewhere, I don't think there is a need to backport these. Ref: https://mail.openvswitch.org/pipermail/ovs-dev/2023-January/401584.html > > Anyway, LGTM: > > Acked-by: Ilya Maximets <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
