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

RE: svn commit: r1729060 - /subversion/trunk/tools/client-side/svn-graph.pl

From: Bert Huijben <bert_at_qqmail.nl>
Date: Tue, 9 Feb 2016 12:47:15 +0100

> -----Original Message-----
> From: jamessan_at_apache.org [mailto:jamessan_at_apache.org]
> Sent: maandag 8 februari 2016 01:40
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1729060 - /subversion/trunk/tools/client-side/svn-
> graph.pl
>
> Author: jamessan
> Date: Mon Feb 8 00:39:43 2016
> New Revision: 1729060
>
> URL: http://svn.apache.org/viewvc?rev=1729060&view=rev
> Log:
> * tools/client-side/svn-graph.pl
> (): Use direct method invocation to construct SVN::Client
> (parse_commandline): Canonicalize the given repo URI
> (write_graph_descriptor): Use the canonical form of the relpath for the root
> of the repo.
>
> Modified:
> subversion/trunk/tools/client-side/svn-graph.pl
>
> Modified: subversion/trunk/tools/client-side/svn-graph.pl
> URL: http://svn.apache.org/viewvc/subversion/trunk/tools/client-side/svn-
> graph.pl?rev=1729060&r1=1729059&r2=1729060&view=diff
> ==========================================================
> ====================
> --- subversion/trunk/tools/client-side/svn-graph.pl (original)
> +++ subversion/trunk/tools/client-side/svn-graph.pl Mon Feb 8 00:39:43
> 2016
> @@ -62,7 +62,7 @@ parse_commandline();
>
> # Point at the root of a repository so we get can look at
> # every revision.
> -my $auth = (new SVN::Client())->auth;
> +my $auth = SVN::Client->new->auth;

Technically this auth instance has the same lifetime as this SVN::Client. I don't know how this is exposed in the perl code (and if that lifetime matches that of the application), but it would be cleaner to also keep the client instance in a different variable.

> my $ra = SVN::Ra->new(url => $repos_url, auth => $auth);

And there are helper functions on the client to create an Ra session from that, which automatically hook a few other features that are now ignored.

        Bert
Received on 2016-02-09 12:47:24 CET

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