Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Convert to forward slash before creating leading directory #80

Closed
wants to merge 1 commit into from
Closed

Convert to forward slash before creating leading directory #80

wants to merge 1 commit into from

Conversation

linquize
Copy link

Fix clone failure when target directory is C:\1\2 but C:\1 does not exist, using Windows Command Prompt

Fix clone failure when target directory is C:\1\2 but C:\1 does not exist, using Windows Command Prompt
@buildhive
Copy link

MSysGit - the development behind Git for Windows » git #109 SUCCESS
This pull request looks good
(what's this?)

@dscho
Copy link
Member

dscho commented Dec 17, 2013

Sorry it took so long for somebody to look over this... but... would it not make more sense to teach safe_create_leading_directories to handle backslashes on Windows, too? I.e. something like

while (*pos && !is_dir_sep(*pos))
        pos++;
if (!*pos)
        break;

instead of

pos = strchr(pos, '/');
if (!pos)
        break;

at https://github.com/linquize/msysgit-git/blob/869e795d8f370386d1c63366f964ed27765d5a6a/sha1_file.c#L111

@sschuberth
Copy link
Member

Indeed @dscho's proposal looks more elegant. Any objection that I try to take this upstream?

@dscho
Copy link
Member

dscho commented Dec 18, 2013

@sschuberth Do go ahead!

@dscho
Copy link
Member

dscho commented Dec 30, 2013

Since the proposed solution is superseded by my suggestion -- and since it is unclear that this change is needed in the first place -- I am closing this bug.

@dscho dscho closed this Dec 30, 2013
@linquize
Copy link
Author

So, is the issue fixed?

@dscho
Copy link
Member

dscho commented Dec 31, 2013

@linquize no. It did not look to me as if you were still interested in it, so I closed it, especially after you did not react to my suggestion and after @sschuberth (who might, or might not, have time to work on it) agreed that the solution I proposed would be better.

@sschuberth
Copy link
Member

The upstream patch / discussion is at http://www.spinics.net/lists/git/msg222459.html.

@linquize linquize deleted the slash-create-leading-dir branch February 24, 2015 14:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants