> On Thu, Oct 10, 2013 at 5:49 PM, Bob Archer <bob.arc...@amsi.com> wrote:
> I assume he was asking how to "fix" the blame. Cause, sure, he could open
> the file, convert it back to UTF-8 with CRLF line endings... and commit it... 
> of
> course, now blame is going to show him on every line, since he just changed
> every line.
> 
> That's exactly what I meant.  You're correct with how the blame is handled.  I
> committed the UTF-8 copy to a test branch, diff'd, and it showed every line
> as being changed.  Unfortunately it looks like this is our best option.

Yep, we have done the same thing. As a matter of fact, I just over the past few 
days rescripted all our database scripts to be UTF-8 since merging them just 
doesn't work correctly when they are UTF-16 even if you remove the binary mime 
type.

> 
> 
> On Thu, Oct 10, 2013 at 7:07 PM, Ben Reser <b...@reser.org> wrote:
> At current blame is not UTF-16 aware.

It's not just blame that isn't... the diff engine, or whatever detects file 
types always considers UTF-16 files to be binary. If you "add" a UTF-16 file 
you see that svn adds the application/octet-stream mime type.  There is an 
issue in the bug database about this from when I reported/complained about 
it... however it hasn't been addressed. I'm surprised still at this time that 
svn still can't support UTF-16 text files as text wrt adding, diffing, blaming, 
etc.

BOb


> 
> About a year ago there was a "patch" (actually they just reposted an entire
> copy of blame.c) posted that helped, but it really didn't go anywhere since
> the original poster didn't continue following up.
> 
> Perhaps you'll find the above useful.  Patches are of course welcome.
> 
> I'll take a look at that this weekend.  Thanks for all the input everyone.
> 
> 
> On Thu, Oct 10, 2013 at 7:07 PM, Ben Reser <b...@reser.org> wrote:
> On 10/10/13 12:17 PM, T.J. Perovich wrote:
> > On Thu, Oct 10, 2013 at 2:27 PM, Ryan Schmidt
> > <subversion-20...@ryandesign.com <mailto:subversion-
> 20...@ryandesign.com>> wrote:
> >>Sounds like you've converted the file from UTF-8 to UTF-16.
> >
> > Thanks, you're absolutely right.  It changed from UTF-8 to UTF-16LE.
> >
> > Any idea how to go about fixing it elegantly?  We have about 3 months
> > of commits since this happened.  Diff's in the GUIs have been working
> > fine and we don't blame too often which is why it was never noticed.
> At current blame is not UTF-16 aware.
> 
> About a year ago there was a "patch" (actually they just reposted an entire
> copy of blame.c) posted that helped, but it really didn't go anywhere since
> the original poster didn't continue following up.
> 
> https://mail-archives.apache.org/mod_mbox/subversion-
> dev/201207.mbox/%3CCAAF0CB13B282344AF95AD2DE3D1962215627A3C%40
> DAG-B.nexon.corp%3E
> 
> and the followup
> 
> https://mail-archives.apache.org/mod_mbox/subversion-
> dev/201208.mbox/%3CCAB84uBVVrHFfQyEA5pF5gStMpXz+RH2jKvdvCQsCO
> cjv+rq...@mail.gmail.com%3E
> 
> Perhaps you'll find the above useful.  Patches are of course welcome.

Reply via email to