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

Re: [PATCH]remove adm_access in svn_wc_translated_file3

From: Branko Cibej <brane_at_xbc.nu>
Date: Fri, 07 Aug 2009 13:00:34 +0200

Stefan Sperling wrote:
[...]
> Yes! Please run *all* tests before submitting a patch.
>
[...]
> The test suite also takes about an hour to run for me. But as long as
> it is just my computer spending the time and not myself, I don't really
> care. I can do something else while the tests are running, e.g. read
> mail or put up the laundry or something.
>

When you're *fixing* test failures, it's perfectly find to run just
individual tests, or small subsets. But definitely run the whole
testcase before submitting a patch, yes.

-- Brane

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2381249
Received on 2009-08-07 13:01:02 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.