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

Re: Sparse checkouts suggestion

From: Branko Čibej <brane_at_apache.org>
Date: Thu, 14 Sep 2017 14:38:57 +0200

On 14.09.2017 12:26, Paul Hammant wrote:
> Thanks. 
> Here's what I expect to land - https://github.com/apache/subversion/pull/5

Why o why did you go to the trouble of adding the global variables in
svn-viewspec.py? They add exactly zero value and cause nothing but code
churn. -1 to that change ... the rest there look cosmetic and OK.

BTW, you left trailing spaces on some of the lines (github diff shows
them); please fix that.

The tests should be in tools/client-side. subversion/tests/cmdline are
for testing the main command-line tools, and have their own test
infrastructure (which predates PyTest, FWIW). We do not install, nor
officially support, the bits in the tools/ directory.

Also please read the community guide, especially the part about log
messages, before committing.

> Ten tests - ten passing in less than 0.1s
> I don't know if there is a clean workflow from Git commits back into
> Subversion so, I'll make the same commit through the Svn HTTPS
> interface, if code review comments are approving.

There isn't currently, as far as I know.

-- Brane
Received on 2017-09-14 14:39:02 CEST

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