Hi Tom, On Wed, Oct 9, 2013 at 9:29 AM, Tom Rini <[email protected]> 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. Regards, Simon
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
