[U-Boot] [PATCH 2/4] Coding Style cleanup: replace leading SPACEs by TABs

Tom Rini trini at ti.com
Wed Oct 9 17:39:14 CEST 2013


On Wed, Oct 09, 2013 at 09:33:53AM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On Wed, Oct 9, 2013 at 9:29 AM, Tom Rini <trini at ti.com> wrote:
> 
> > On Wed, Oct 09, 2013 at 08:02:10PM +0900, Masahiro Yamada wrote:
> > > Hello, Wolfgang, Simon.
> > >
> > > (I added Cc Simon)
> > >
> > >
> > > >  tools/buildman/README                              |  268 +--
> > > >  tools/buildman/board.py                            |  236 +-
> > > >  tools/buildman/bsettings.py                        |   18 +-
> > > >  tools/buildman/builder.py                          | 2374
> > ++++++++++----------
> > > >  tools/buildman/buildman.py                         |   16 +-
> > > >  tools/buildman/control.py                          |  136 +-
> > > >  tools/buildman/test.py                             |  162 +-
> > > >  tools/buildman/toolchain.py                        |  392 ++--
> > >
> > > I might be too worried, but so many lines are going to be changed.
> > > Just in case, I hope Simon checks this.
> >
> > I'm not sure whacking the whitespace in quasi-external (I know patman is
> > used outside U-Boot, I forget about buildman) python tools is the right
> > way to go here.  Today the tools are consistent and space-only.  This
> > makes it mix-and-match.
> 
> patman also. The coding style I tried to follow for Python is PEP 8 - with
> spaces. At least it is consistent.

OK, yeah.  Wolfgang, please drop patman/buildman and
test/image/test-fit.py changes out of this.  reformat.py is consistently
using tabs, except for what you fixed, so that's fine.

If that's still a problem, we need to reformat the whole of the tool to
something tab-consistent instead, entirely.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131009/4c42c759/attachment.pgp>


More information about the U-Boot mailing list