-
Notifications
You must be signed in to change notification settings - Fork 2.5k
clone: don't test remote url against filesystem #5983
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
Conversation
The test from #5950 could be expanded to also include stat, but git_path_exists and git_path_isdir doesn't specifically check errno and have no way of returning invalid path rather than non-existing path so it doesn't seem like a good option here. |
Yeah, Windows is going to be the tricky bit here for sure. Let me do some testing over there. |
Would it be possible to add |
An alternative to checking the url for "://" could be to split up |
I figured that the way to test this is to add a "http:/"-folder in the current directory for the online cloning tests. We can't really test this on Windows systems, but then again, it will never occur on those either. |
Restructured and added comments to make the flow clearer. |
resolves #5982
I removed the directory check in create_and_configure_origin
If the path given is to a bundle it will not be a directory but should probably have an absolute path anyway?
No test added, I'm not sure about the best approach to test this.
I suspect windows will act up on url's containing ':'