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