Re: [PATCH 0/2] Move push logic to transport.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 9 Mar 2009, Johannes Schindelin wrote:

> Hi,
> 
> On Mon, 9 Mar 2009, Daniel Barkalow wrote:
> 
> > On Mon, 9 Mar 2009, Johannes Schindelin wrote:
> > 
> > > On Mon, 9 Mar 2009, Daniel Barkalow wrote:
> > > 
> > > > On Mon, 9 Mar 2009, Johannes Schindelin wrote:
> > > > 
> > > > > On Mon, 9 Mar 2009, Daniel Barkalow wrote:
> > > > > 
> > > > > > On Mon, 9 Mar 2009, Johannes Schindelin wrote:
> > > > > > 
> > > > > > > On Sun, 8 Mar 2009, Daniel Barkalow wrote:
> > > > > > > 
> > > > > > > > It doesn't convert http-push or the rsync transports, largely 
> > > > > > > > because I don't have test setups for rsync or webdav to make sure 
> > > > > > > > that they're still working.
> > > > > > > 
> > > > > > > $ ls t/*http-push*
> > > > > > > t/t5540-http-push.sh
> > > > > > > 
> > > > > > > $ git grep -n test.*rsync t/
> > > > > > > t/t5510-fetch.sh:195:test_expect_success 'fetch via rsync' '
> > > > > > > t/t5510-fetch.sh:206:test_expect_success 'push via rsync' '
> > > > > > > t/t5510-fetch.sh:217:test_expect_success 'push via rsync' '
> > > > > > > 
> > > > > > > It should be just a matter of installing an apache and rsync.
> > > > > > 
> > > > > > And configuring them suitably, yes. That's the part I haven't previously 
> > > > > > done.
> > > > > 
> > > > > If you have to configure apache (or rsync) for the test to run properly, 
> > > > > we have a serious bug in our test suite.  Please share the output in that 
> > > > > case.
> > > > 
> > > > rsync: failed to connect to 127.0.0.1: Connection refused (111)
> > > > rsync error: error in socket IO (code 10) at clientserver.c(104) [receiver=2.6.9]
> > > > fatal: Could not run rsync to get refs
> > > > 
> > > > I mean, I won't need to do anything really special in the configuration, 
> > > > but I do need to have rsyncd running giving access to that directory 
> > > > without any funny mapping on localhost.
> > > 
> > > That is my mistake, I guess.  I tried to be lazy and not change 
> > > transport.c so that it groks "rsync:$(pwd)" style paths.
> > 
> > Oh, I think it's probably not worth allowing rsync:$(pwd) (particularly 
> > since rsync itself seems not to let you say "rsync:" for local paths). 
> 
> This contradicts that:
> 
> > I'll set something up to run rsyncd with an appropriate config, like the 
> > lib-httpd stuff. Oh, and it looks like nobody actually runs the rsync 
> > tests; they're broken by $(pwd) including whitespace, and haven't been 
> > fixed yet.
> 
> If we would support local rsync paths, testing it would be easy.

Ok, I think having transport_get strip off the "rsync:" would make it work 
with the rest of the functions unchanged, which would in turn mean that 
the tests would be a pretty reasonable approximation of the real-world 
code.

	-Daniel
*This .sig left intentionally blank*
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux