> -----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-
> Author: jamessan
> Date: Mon Feb 8 00:39:43 2016
> New Revision: 1729060
> URL: http://svn.apache.org/viewvc?rev=1729060&view=rev
> * 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
> URL: http://svn.apache.org/viewvc/subversion/trunk/tools/client-side/svn-
> --- subversion/trunk/tools/client-side/svn-graph.pl (original)
> +++ subversion/trunk/tools/client-side/svn-graph.pl Mon Feb 8 00:39:43
> @@ -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.
Received on 2016-02-09 12:47:24 CET