[ovs-dev] [PATCH 12/13] treewide: Convert leading tabs to spaces.

Ben Pfaff blp at ovn.org
Mon Jun 11 22:38:02 UTC 2018


On Mon, Jun 11, 2018 at 02:47:41PM -0700, Justin Pettit wrote:
> 
> 
> > On Jun 4, 2018, at 2:27 PM, Ben Pfaff <blp at ovn.org> wrote:
> > 
> > It's always been OVS coding style to use spaces rather than tabs for
> > indentation, but some tabs have snuck in over time.  This commit converts
> > them to spaces.
> > 
> > Signed-off-by: Ben Pfaff <blp at ovn.org>
> > ---
> > ...
> > vswitchd/vswitch.xml                              |    2 +-
> > xenserver/openvswitch-xen.spec.in                 |    2 +-
> > 69 files changed, 3372 insertions(+), 3372 deletions(-)
> > mode change 100755 => 100644 debian/openvswitch-switch.init
> > mode change 100755 => 100644 debian/ovn-central.postinst
> > mode change 100755 => 100644 debian/ovn-host.postinst
> > mode change 100755 => 100644 rhel/etc_init.d_openvswitch
> > mode change 100755 => 100644 rhel/etc_sysconfig_network-scripts_ifdown-ovs
> > mode change 100755 => 100644 rhel/etc_sysconfig_network-scripts_ifup-ovs
> > mode change 100755 => 100644 tests/run-oftest
> > mode change 100755 => 100644 tests/run-ryu
> > mode change 100755 => 100644 utilities/bugtool/ovs-bugtool-tc-class-show
> > mode change 100755 => 100644 utilities/ovs-sim.in
> 
> Did you mean to remove the execute bit from these files?

Oops, thanks for pointing that out.  I fixed it.

> Thanks for making these changes.  For the patches up to this point:
> 
> Acked-by: Justin Pettit <jpettit at ovn.org>

Thanks for the review!


More information about the dev mailing list