Re: compile failure and other trouble

On Sat, Aug 11, 2001 at 09:14:29AM +0200, Cynic wrote:
> Hi there,
>
> 1)
>
> --------------------Configuration: svn_com - Win32 Release--------------------
> Creating Type Library...
> [...]
> D:\compile\subversion\subversion\clients\win32\svn_com\SVN.cpp(34) : error C2668: 'InlineIsEqualGUID' : ambiguous call to overloaded function
> D:\compile\subversion\subversion\clients\win32\svn_com\SVN.cpp(314) : error C2660: 'apr_hash_first' : function does not take 1 parameters
> SVNCOM.cpp
> SVNStatus.cpp
> D:\compile\subversion\subversion\clients\win32\svn_com\SVNStatus.cpp(30) : error C2668: 'InlineIsEqualGUID' : ambiguous call to overloaded function
> Error executing xicl6.exe.
>
> __build__ - 3 error(s), 12 warning(s)
>
> apr_hash_first has apparently changed in APR. This is what apr_hash.h has to
> say about it && the proto:
>
> * Start iterating over the entries in a hash table.
> * @param p The pool to allocate the apr_hash_index_t iterator
> * @param ht The hash table
> * @return a pointer to the iteration state, or NULL if there are no entries.
>
> APR_DECLARE(apr_hash_index_t *) apr_hash_first(apr_pool_t *p, apr_hash_t *ht);
>
> I don't know what to do about the other error.

Thanks, fixed the apr_hash_first problem, will have to defer the other problem
to someone else.

>
> 2)
>
> subversion\subversion\libsvn_subr\libsvn_subr.dsp looks for base64.c, but
> that file is svn_base64.c.

Also fixed.

>
> 3)
>
> It's not really necessary to require people to define $(SVN_DB3_WIN32):
> all that you need is put the include/ and lib/ directories in the appropriate
> lists in Tools -> Options -> Directories in MSVC, and you can get rid of those
> /I $(SVN_DB3_WIN32)/include and just "ADD LINK32 libdb32.lib" in the .dsp files.
>
> I can send the patches since I've already done this -- didn't like the idea I had
> to restart my computer just to compile Subversion.

I tend to agree with you, but I'm not sure how others feel. If you want to
send the patches to this list, there is a good chance they will be committed,
unless there is some reason not to.
>
> 4)
>
> Nowhere in the distribution is it note whether you have to build apr and neon
> by hand prior to building subversion or whether they get build automagically.
> But since the subversion workspace doesn't ask for neon.dsp (as opposed to apr.dsp)
> I guess you have to build it yourself.
>
I thought that at the moment, we didn't use neon in the win32 build, which is
why it is not mentioned, (although maybe it still should be). My impression
was that currently the client only works locally on win32.

> 5)
>
> This is not your problem, but anyway: neon-15.3 (the one at ProjectDownloadList)
> doesn't build. Can't try with anything newer, http://www.webdav.org/neon/ doesn't
> respond.

Well, since gstein provides the hosting for webdav.org, and several neon
committers are subscribed to this list, it is not too far off-base. But we are
aware of that.

Thanks for the catches.

-- 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Kevin Pilch-Bisson                    http://www.pilch-bisson.net
     "Historically speaking, the presences of wheels in Unix
     has never precluded their reinvention." - Larry Wall
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~