From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 9 Oct 2013 11:39:14 -0400 Subject: [U-Boot] [PATCH 2/4] Coding Style cleanup: replace leading SPACEs by TABs In-Reply-To: References: <1380901406-19303-1-git-send-email-wd@denx.de> <1380901406-19303-3-git-send-email-wd@denx.de> <20131009200209.5574.AA925319@jp.panasonic.com> <20131009152925.GI15917@bill-the-cat> Message-ID: <20131009153914.GJ15917@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 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: