At 12:44 PM 7/17/2002, =?UTF-8?B?QnJhbmtvIMSMaWJlag==?= wrote:
>Ben Collins-Sussman wrote:
>
>>Bill Rowe is changing the entire apr_xlate implementation right now;
>>if you try to use the latest APR, then subversion won't work at all if
>>build it with --enable-utf8.
>>
>>His solution so far:
>>
>> * move the apr_xlate API into apr-iconv.
>Yeah, and he forgot to migrate the configury that defines HAVE_ICONV_H and
>HAVE_ICONV in the private header from apr to apr-util. _That_ is why
>apr_xlate_* doesn't work now; the transition was not complete.
Let me please clarify, before people start SCREAMING, that the apr_xlate API
moved to apr-util. It will either use the installed iconv, or our own
apr-iconv
which itself requires the apr portability API.
As far as APR_HAVE_ICONV_H and APR_HAVE_ICONV, those are misplaced
right now. I'm asking for help with the library detection. We need to
determine
this upfront, in apr, so we can decide if apr-iconv or the platform iconv
aught to
be used.
>>Of course, Subversion's build system knows nothing about compiling
>>apr-iconv, even if it's sitting right next to apr and apr-util. So
>>whenever one of the svn_utf_* functions are called, apr returns an
>>error, and so does svn.
>
>The APR configury should still find the installed (native) iconv on Unix.
>That must _not_ change. I'd still argue for some form of my drop-in iconv
>support on Windows, because I can't for the life of me see how to get
>apr-iconv actually working on Windows in a day or two.
This evening, in fact, we will be building apr-iconv for the win32 build of
apr-util.
We need to revisit all of the libdb and libiconv issues for the win32
platform. We have no unix-like standards that dictate where or how
such things are shared globally on a win32 box. But I'm not about
to commit undocumented gross hacks to force this all to work around
one installation.
openssl, iconv, db, ldap; all of these need to be revisited.
>>For now, the solution is to make sure your apr is at least a couple
>>days old. I just did a "cvs up -dP -D 'last week'", and that seems to
>>work fine.
>
>Right. APR HEAD is broken. The changes were made yesterday, so Monday's
>APR and APR-util should be just right.
Yes. Also, if you have a checkout from mid-last-night, I had missed
one line changes in both configure.in scripts for apr and apr-util that
weren't committed until about 0715Z.
Bill
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jul 17 20:33:25 2002