[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Standardizing on UTF8 internally isn't enough

From: Justin Erenkrantz <justin_at_erenkrantz.com>
Date: 2007-07-19 04:21:54 CEST

On 7/17/07, Erik Huelsmann <ehuels@gmail.com> wrote:
> This problem could be solved to by adding the ICU lib as a dependency
> and change all path comparisons to use the ICU normalform agnostic
> comparison routines.

I am pretty much against the idea of adding a *required* 39MB (~10MB
gzip'd) library that isn't widely available as a requirement to run
SVN. Yes, Debian/Ubuntu has it as do some others; but it's not that
popular; quick look at the release notes indicates that ICU requires
VC2003+ for compilation - so does it actually support VC6 (listed as
"rarely tested")? We had enough Win32 people revolt against APR-iconv
- now, we're going to backtrack and add in a *huge* library to replace
it? Yikes. This is like squashing a fly with a sledgehammer, IMO.

Why can't we just do input validation ourselves? This is clearly a
very specific corner case and one we can detect quite trivially (i.e.
look for the chars that only exist in normalization form D).

My $.02. -- justin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jul 19 04:21:04 2007

This is an archived mail posted to the Subversion Dev mailing list.