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

Re: [PATCH] wc-replacements branch: Implement reverting after svn cp URL PATH replacements

From: <kfogel_at_collab.net>
Date: 2005-09-20 04:05:55 CEST

Ivan Zhakov <chemodax@gmail.com> writes:
> > There's one problem with this patch: merge_tests.py 26 fails after
> > applying it. I'm investigating why, but in case I don't resolve the
> > issue tomorrow, I thought I'd let you know.
> Oops, on my machine it fails too. I look why is it. Seems I missed
> that svn_wc_add_repos_file called also from merge_file_added().

Ivan, you've been doing terrific work on the wc-replacements branch,
so please don't take this comment as a condemnation:

If the test fails on your machine too, then you can know that before
posting the patch :-). In other words, always run the test suite on a
change before posting the change. No point spending a reviewer's time
on something that can be automatically discovered earlier.

You probably came to this conclusion independently. I just wanted to
confirm that it is standard practice for all of us.


www.collab.net  <>  CollabNet  |  Distributed Development On Demand
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Sep 20 05:12:36 2005

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.