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

RE: svn commit: r1704847 - /subversion/trunk/subversion/libsvn_ra_serf/serf.c

From: Bert Huijben <bert_at_qqmail.nl>
Date: Wed, 23 Sep 2015 15:42:29 +0200

> -----Original Message-----
> From: Daniel Shahaf [mailto:danielsh_at_apache.org]
> Sent: woensdag 23 september 2015 15:39
> To: Ivan Zhakov <ivan_at_visualsvn.com>
> Cc: dev_at_subversion.apache.org
> Subject: Re: svn commit: r1704847 -
> /subversion/trunk/subversion/libsvn_ra_serf/serf.c
>
> Ivan Zhakov wrote on Wed, Sep 23, 2015 at 16:32:15 +0300:
> > On 23 September 2015 at 16:29, <danielsh_at_apache.org> wrote:
> > > Author: danielsh
> > > Date: Wed Sep 23 13:29:30 2015
> > > New Revision: 1704847
> > >
> > > URL: http://svn.apache.org/viewvc?rev=1704847&view=rev
> > > Log:
> > > * subversion/libsvn_ra_serf/serf.c
> > > (ra_serf_version): Report both built- and run-time versions of
libserf,
> > > like we do for everything else.
> > >
> > May be we should report them separately only if built and runt-time
> > versions are different?
>
> We unconditionally report both versions for everything. (And I think
> that's fine.)

Perhaps we should move the serf version to the standard list of linked
dependencies and do something smarter there.

Now it is just extending an ra implementation description, while ra_svn
doesn't report the version of Cyrus Sasl at all.

        Bert
Received on 2015-09-23 15:42:56 CEST

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