[MERGE] alias to respect spaces in quoted arguments (version 3)
Marius Kruger
amanic at gmail.com
Mon Dec 18 23:47:55 GMT 2006
On 12/12/06, Aaron Bentley <aaron.bentley at utoronto.ca> wrote:
>
> Aaron Bentley has voted +1.
> Status is now: Approved
> Comment:
> This has quite a lot of spurious whitespace changes. Please try to avoid
> that in the future. I'll fix them as part of the merge.
>
> For details, see:
> http://bundlebuggy.aaronbentley.com/request/%3C418c22640611292054t7aed8ca7p4fac2db426677ea2%40mail.gmail.com%3E
>
> thanks, for the vote and the merge.
sorry about the whitespace (my editor removes it automatically),
I would have reverted that, but I remembered something on the mailinglist
about removing all the trailing whitespace, so I removed them.
I tried to see if you fixed this, but it looks like you forgot.
----some previous post----
> I think it is fine. And I think we are happy to get the code cleaned up.
> If you want some more things to look for, we also need to remove
> trailing whitespace from lines. Though that may be a more controversial
> change, since it messes up annotations.
>
> John
> =:->
>
Some notes:
1.) Can we add something to the branch to specify that for certain or all
files we should
a.) automatially remove trailing whitespace or
b.) refuse to commit it.
So that we don't get into this mess. This might have been descussed
before. It could be implemented as a plugin.
2.) Annotate should be able to cope with this. mabye add an option to ignore
whitespace changes.
I often get into a situation where I try to find the true author of a
line (to blame/praise them:),
but sometimes the line might have been tuched by several people in the
past.
I thought the bisect plugin could help with this, but I was never able
to obtain it,
I must probably try again.
--
I code therefore I am.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20061219/49cf1700/attachment.htm
More information about the bazaar
mailing list