[MERGE] whitespace normalization

Ben Finney bignose+hates-spam at benfinney.id.au
Fri Jul 25 00:35:44 BST 2008


Aaron Bentley <aaron at aaronbentley.com> writes:

> Aaron Bentley has voted reject.
> Status is now: Vetoed
> Comment:
> Changes like this cause spurious conflicts.

As do differences between files caused by *not* normalising the
whitespace in the code base.

If the code has been allowed to become inconsistent with whitespace,
it seems the only long-term solution is to declare an
automaticaly-checkable policy, convert the code base, and reject any
future commit that doesn't conform.

Would Benjamin's patch be better if a whitespace policy (such as
"conform with PEP 8 whitespace suggestions", or even "ensure
whitespace is as produced by reindent.py") were implemented, before a
one-time fixup to the code base?

-- 
 \        “The right to search for truth implies also a duty; one must |
  `\      not conceal any part of what one has recognized to be true.” |
_o__)                                                 —Albert Einstein |
Ben Finney




More information about the bazaar mailing list