[MERGE] integrated EOL conversion support

Ian Clatworthy ian.clatworthy at internode.on.net
Sun Mar 29 16:30:02 BST 2009


John Arbash Meinel wrote:

> I'm still a bit concerned that the labels are the "wrong way" around. In
> that you are setting the line-endings in the repository, rather than
> settting the line-endings in the working tree. I'm pretty sure other
> systems like svn have the value indicate the working tree line endings.
> 
> (At least last I checked, 'crlf' meant native locally and crlf in the
> repo, rather than whatever in the repo and crlf when I'm working with it.)

I'm going to wait a day or two before landing this because I want to make
sure everyone has a chance to raise concerns like this. This is an important
feature and like many things, it makes a statement about what Bazaar stands
for: something cleanly designed and well thought through, something far
less complex than git but more flexible/powerful than Mercurial. (Ducks
quickly.) :-)

> I honestly think people care more about the value they get when they are
> working on the file, because it matters what other tools are going to do
> with it.
> 
> I'm certainly not trying to block, just address a possible point of
> confusion.

And I understand your point. The problem is that some/many projects will
define project-specific rules and check them in to their branch or
version-control them in a separate, shared location that developers will
be told to point their path to. The moment I deliver custom rules - rules
defined somewhere else besides each user's bazaar.conf file, then the
emphasis *must* be first-and-foremost on what get's *committed*, not what
individual users would like in their working tree. I also
feel that native-in-the-tree is the most sensible default so specifying
that is a bit redundant.

I'll *gladly* take suggestions for better names and I'm genuinely happy
to change them if we can get some consensus on the list about better names
than what I've put forward.

Ultimately, I think it will come down to how good the help is.
For those who have not looked inside the patch, the help is attached.

Ian C.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: eol.txt
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20090330/52d59c76/attachment-0001.txt 


More information about the bazaar mailing list