hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ted Yu <yuzhih...@gmail.com>
Subject removing trailing whitespace in patch
Date Sat, 27 Aug 2011 13:51:01 GMT
Hi,
A big portion of review is on removing trailing whitespace in patch.
I think contributors can help reduce the amount of time spent in this
trivial task.
The following description is for Mac OS X, but it is easy to customize for
your OS. See
http://stackoverflow.com/questions/149057/how-to-removing-trailing-whitespace-of-all-files-recursively

1. Generate patch as usual (e.g. 3900.addendum)
2. Use the following command on patch:
sed -i .bak -E 's/ *$//g' 3900.addendum
3. The above operation would remove trailing whitespaces on lines not
changed by the patch. So committer needs to use --ignore-whitespace option
when applying the patch.

Your comments/ideas are welcome.

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message