Julian Foad <julianfoad@btopenworld.com> writes:
> I am voting -1 for this, due to the "repair" of line endings. For
> the other reasons mentioned below I would have voted "-0".
>
> A normal commit, via svn_wc_transmit_text_deltas, does not force
> repair of inconsistent line endings, but fails with an error. This
> patch to "import", on the other hand, silently "repairs"
> inconsistent line endings. I think that is the wrong thing for it
> to do, as I don't see any reason for it to behave differently from
> "commit".
>
> This patch is on the fuzzy line between fix and enhancement. Lack
> of this patch only "allows malformed data" in the same way that any
> commit does if "svn:eol-style" and "svn:keywords" are not as
> intended. Behaviour without this patch is to preserve the file
> contents exactly. For these reasons I don't think there is a strong
> need to put this patch in 1.0.1.
I voted +1 for the change, but still have to say your argument is
persuasive...
Next question is: does the change belong in 1.1? Do you consider the
behavior undesirable overall, or just not appropriate for 1.0.1?
-K
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Mar 12 02:25:39 2004