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

Re: [PATCH] serf backend requires servers category configuration

From: Kouhei Sutou <kou_at_cozmixng.org>
Date: Tue, 1 Apr 2008 08:53:13 +0900

Hi,

2008/4/1, Lieven Govaerts <svnlgo_at_mobsol.be>:
> Silly question: what does svn do without your patch? Does it crash?
> Ignore configuration?

I don't know about svn but a Ruby script showed in
http://svn.haxx.se/dev/archive-2008-03/0933.shtml
is crashed.

> $ cat svn_test.rb
> require 'svn/ra'
> context = Svn::Client::Context.new
> callbacks = Svn::Ra::Callbacks.new(context.auth_baton)
> session = Svn::Ra::Session.open("http://localhost/", {}, callbacks)
> session.commit_editor2("commit log") {}

Thanks,

--
kou
>  Kouhei Sutou wrote:
>  > Hi,
>  >
>  > I found a difference between Neon backend and Serf backend.
>  >
>  > Neon backend doesn't require SVN_CONFIG_CATEGORY_SERVERS
>  > configuration but Serf backend does.
>  >
>  > Index: subversion/libsvn_ra_serf/serf.c
>  > ===================================================================
>  > --- subversion/libsvn_ra_serf/serf.c  (revision 30128)
>  > +++ subversion/libsvn_ra_serf/serf.c  (working copy)
>  > @@ -374,9 +374,12 @@
>  >                   SVN_CONFIG_OPTION_SSL_AUTHORITY_FILES, NULL);
>  >  #endif
>  >
>  > -  server_group = svn_config_find_group(config,
>  > -                                       session->repos_url.hostname,
>  > -                                       SVN_CONFIG_SECTION_GROUPS, pool);
>  > +  if (config)
>  > +    server_group = svn_config_find_group(config,
>  > +                                         session->repos_url.hostname,
>  > +                                         SVN_CONFIG_SECTION_GROUPS, pool);
>  > +  else
>  > +    server_group = NULL;
>  >
>  >    if (server_group)
>  >      {
>  >
>  >
>  > Thanks,
>  > --
>  > kou
>  >
>
> > ---------------------------------------------------------------------
>  > To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
>  > For additional commands, e-mail: dev-help_at_subversion.tigris.org
>  >
>  >
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-01 01:53:23 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.