[Buildroot] svn commit: trunk/buildroot/toolchain/binutils
Thiago A. Corrêa
thiago.correa at gmail.com
Tue Aug 21 13:48:35 UTC 2007
Well, if someone is willing to do a quick script to check the
whitespace stuff, I would say let him. It's not like one is loosing
anything, and it makes it easier to make sure. Though one can be
careful enough to avoid the whitespace issue, automated checks (and
checks alone, not fixups! ) is A Good Thing (tm) imho. It could even
be used in SVN hooks to deny commits that are invalid, but I'm not
quite sure if it's possible to apply it just to Makefiles and *.mk.
Just my 2 euro cents :)
On 8/21/07, Bernhard Fischer <rep.dot.nop at gmail.com> wrote:
> On Mon, Aug 20, 2007 at 08:18:46PM +0200, Cristian Ionescu-Idbohrn wrote:
> >On Mon, 20 Aug 2007, Ulf Samuelsson wrote:
> >> Thanks, maybe we should have a script which is periodically run to find
> >> the white space stuff and remove it.
> >Easiest is to avoid putting it there, IMHO.
> Agree. Not adding code that breaks the established style is the proper
> thing to do. Automated fixup is not a solution nor desirable. It's not
> that avoiding silly whitespace damage should be _that_ hard (modulo in
> an X11 impl or the like ;).
> buildroot mailing list
> buildroot at uclibc.org
More information about the buildroot