EOL support trouble in 1.15
Ian Clatworthy
ian.clatworthy at internode.on.net
Mon Jun 1 08:50:10 BST 2009
Frits Jalvingh wrote:
> After some debugging I found out that not only the shared repo needs a proper
> format but the branched repo needs a higher format too... It used working tree
> format 4 while at least 5 was needed... That was not very clear to me (first
> time I ever encountered a working tree format) - it might be a good idea to at
> least put that in the reference documentation.
Yes, that's really annoying. I'll update the documentation.
> It might also be a good idea to at least show a warning when someone tries to
> use EOL filtering while the underlying format(s) do not support it - it now
> failed without any indication and that was hard to debug; I only found out
> about this after lavishly sprinkling print statements thru the bazaar source
> code ;-)
I'll consider this. Until we support branch-specific rules though, my
main concern is that the warning will become really annoying because the
rules need to be defined "globally" for all branches on your computer.
> OK, now filtering works but I have something unexpected. I'm running on Linux,
> and all of my filtered files have been commited using lf eols (in the
> repository).
>
> So, as a test, I added eol = crlf to the rules file for all files and branched.
> The result was a filtered working tree (files have crlf line endings) but all of
> these files show as "modified". bzr status shows all 8000 of them as modified;
That's clearly a bug. I can reproduce it and I'm in the process of
fixing it now.
> doing a bzr diff on a file shows no changes:
That bit is correct.
> jal at pyramides:~/new2/trunk$ bzr diff to.etc.domui/.classpath
> === modified file 'to.etc.domui/.classpath'
>
> I tried commiting; it commits all of these files
That's wrong as well. The underlying cause is the same thing causing
status to display the wrong results. I'm onto it and hope to have a fix
in the next day or two.
Ian C.
More information about the bazaar
mailing list