[cmake-developers] Development pause for sweeping style changes

Stephen Kelly steveire at gmail.com
Tue Aug 14 05:03:13 EDT 2012


Brad King wrote:
> In the future "git blame" may report one of these sweeping
> commits.  In that case, change the blame line to
> 
>  git blame 7bbaa428~1 -- path/of/file/to/blame.txt

You could also tag this commit somehow. 'pre_style_fixup' ?

> We will let this run through dashboard testing tonight on 'next'.
> If all is well then that will become 'master'.

I have a lot of respect for you guys for doing this. Most other large 
development repos would not even consider such a change 'because it messes 
up the history', which I think is nothing compared to the amount of wasted 
time for everyone in handling the existing whitespace problems in the repo. 
Qt is a horrible mess of trailing whitespace. 

In the future we can point to this to see how much of a problem it really is 
for a large and widely used repo with useful history before and after the 
style change.

Kudos,

Steve.




More information about the cmake-developers mailing list