[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