Re: [PATCH] http-push: push <remote> :<branch> deletes remote branch

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Feb 19, 2008, 13:17 UTC
Message-ID: <alpine.LSU.1.00.0802191313290.30505@racer.site>
In-Reply-To: <20080219125840.GA14049@localhost>

Hi,

On Tue, 19 Feb 2008, Clemens Buchacher wrote:

Show 8 quoted lines

> On Mon, Feb 18, 2008 at 05:34:45PM +0000, Johannes Schindelin wrote:
>
> > Thanks for the patch, I think it good.  Now, if we only had automated 
> > tests... ;-)
> 
> Thank you. Implementing a test should be fairly difficult, since we need 
> at least a WebDAV capable webserver, right? Do you having something in 
> mind?

Actually, thinking about it again, it should not be _that_ difficult. We would need to ship a httpd.conf.in, pick a port (maybe 8111, because that is already used in t5703-daemon.sh), generate httpd.conf and then start Apache with that.

If Apache does not want to start, skip this test. (This might happen due to Apache not being installed, or the DAV modules not being installed, or Apache not finding them, or...)

And of course, we would want to have something like NO_SVN_TESTS (maybe NO_APACHE_TESTS) to be able to skip the tests, since they are pretty expensive (starting Apache and all).

> And yes, 'peer' apparently refers to the local end in this case.

Thanks for the clarification.

Ciao, Dscho