clone: don't test remote url against filesystem#5983
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 ':'