Greg Stein <gstein@lyra.org> writes:
> > I'd have looked into fixes, but wasn't sure whether I should given
> > Karl's pronouncement.
>
> Tweaks like this are quite fine. Karl's simply trying to preclude somebody
> checking in a new mechanism for storing WC information :-)
Well, no, please don't check in even tweaks like this -- unless you're
prepared to remember them, look for them in the new repository when it
goes live, and re-check them in if they're absent.
Because if you happen to check it in after we've rolled the dist
tarball and created the new repository, then your changes can't be
easily included without redoing the import and/or re-rolling the
tarball, which would of course imply re-testing it...
Better to hang onto *any* non-M3 changes and check them in tomorrow.
I mean, it's only a day. :-)
[One might say "Yes, but the whole problem is that I'm working on a
Windows machine, and I won't have a working Subversion without these
fixes." To which I say: "If the fixes really fix it, then you *will*
have a working Subversion client, right?" :-) ]
Thanks,
-K
> >...
> > Compiling...
> > commit.c
> > c:\svn\subversion\subversion\libsvn_ra_dav\commit.c(361) : warning
> > C4090: 'function' : different 'const' qualifiers
> > c:\svn\subversion\subversion\libsvn_ra_dav\commit.c(361) : warning
> > C4022: 'ne_add_response_header_handler' : pointer mismatch for actual
> > parameter 4
>
> No idea here. It seems like it thinks "const char **" cannot be passed as a
> "void *". Bogus, but that might be it. Try adding a void* cast in there to
> see what happens. If it shuts the thing down, then go ahead. (with a comment
> on why the cast exists)
>
> > ....
> > session.c
> > c:\svn\subversion\subversion\libsvn_ra_dav\session.c(146) : warning
> > C4013: 'strcasecmp' undefined; assuming extern returning int
>
> Try adding #include <apr_general.h> to the top of the file. Hopefully, that
> will fix it.
>
>
> If the changes work, then go ahead and check them in.
>
> Cheers,
> -g
>
> --
> Greg Stein, http://www.lyra.org/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:36 2006