Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Freeze cloning from a local repository #215

Closed
sirnewton01 opened this issue Mar 7, 2019 · 6 comments
Closed

Freeze cloning from a local repository #215

sirnewton01 opened this issue Mar 7, 2019 · 6 comments
Labels

Comments

@sirnewton01
Copy link
Collaborator

It would be great to be able to run the t5510-fetch tests against some of the new fetch logic that has been added. Unfortunately, during setup it hangs trying to establish a local connection to a git-upload-pack that is launched to handle the source repository. In particular, it is sitting at parseRemoteInitialConnection().

@sirnewton01 sirnewton01 added the bug label Mar 7, 2019
@sirnewton01 sirnewton01 changed the title Freeze closing form a local repository Freeze cloning from a local repository Mar 7, 2019
@sirnewton01
Copy link
Collaborator Author

@driusan does local cloning work for you? I'm wondering if this is somehow a Mac problem.

@driusan
Copy link
Owner

driusan commented Mar 7, 2019

I'm on my tablet and can't check right now, but if memory serves from last time I tried to run the test I was having the same problem. I think I started trying to debug it and found that a read was blocking after it got to the end of stdin but I don't think I got anywhere trying to find a solution.

@driusan
Copy link
Owner

driusan commented Mar 7, 2019

Is ssh working? They're very similar (except one invokes upload-pack over ssh, one invokes it locally), so that's a good place to start

@driusan
Copy link
Owner

driusan commented Mar 9, 2019

The discussion about this is getting fragmented between duplicate issues.. does #225 fix this?

@sirnewton01
Copy link
Collaborator Author

I'll give it a try once it's merged.

@sirnewton01
Copy link
Collaborator Author

This appears to be working now with #225 merged. There's a freeze when doing a local fetch. I'll raise that as a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants