[Piuparts-devel] whitespace cleanup

[Andreas Beckmann]

On 2011-11-08 15:20, Holger Levsen wrote:
>> sed -i -e 's/ *$//' $(find . -name .git -prune -o -type f -print)
> 
> not done. (to big diff, to little value.)

OK, what about

sed -i -e 's/^ *$//' $(find . -name .git -prune -o -type f -print)

That just removes whitespace from whitespace-only lines which are 
especially irritating IMO.

 ChangeLog                       |   38 ++++++------
 NEWS                            |   40 +++++++-------
 README.txt                      |   30 +++++-----
 piuparts-analyze.py             |    4 +-
 piuparts-master.py              |    6 +-
 piuparts-report.py              |    6 +-
 piuparts-slave.py               |   24 ++++----
 piuparts.py                     |  118 +++++++++++++++++++-------------------
 piupartslib/conf.py             |    2 +-
 piupartslib/dependencyparser.py |   44 +++++++-------
 piupartslib/packagesdb.py       |   40 +++++++-------
 unittests.py                    |   14 ++--
 12 files changed, 183 insertions(+), 183 deletions(-)

That's much less than the previous approach to delete *all* trailing whitespace:
 22 files changed, 507 insertions(+), 507 deletions(-)


What about adding "modelines" for the common editors to prevent 
reintroduction of tabs? E.g. for vim I'd use
  # vi:set et ts=4 sw=4 :
in the python scripts.

Andreas