git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] safe_create_leading_directories(): on Windows, \ can separate path components

From
Sebastian Schuberth <sschuberth@gmail.com>
Date
Jan 19, 2014, 20:26 UTC
Message-ID
<CAHGBnuMmutYqwZNw+VoZi=R_LaS_fowZrqaU2+JMYN53q6XQ2Q@mail.gmail.com>
In-Reply-To
<1390088444-11439-1-git-send-email-mhagger@alum.mit.edu>
On Sun, Jan 19, 2014 at 12:40 AM, Michael Haggerty <mhagger@alum.mit.edu> wrote:
Show 5 quoted lines
> This patch applies on top of v3 of mh/safe-create-leading-directories.
>
> The only logical change from Sebastian's patch is that this version
> restores the original slash character rather than always restoring it
> to '/' (as suggested by Junio).

Thanks Michael. This is very similar to Junio's current merge conflict resultion in pu between your original series and my patch (c2fec83). I like this patch of yours slightly better, however, as it uses a "while" instead of a "for" loop and the more conclusive "slash_character" than the "was_slash" variable name.

-- 
Sebastian Schuberth
Previous: Michael HaggertyNext: Junio C Hamano
Message 2 of 3 in “safe_create_leading_directories(): on Windows, \ can separate path components”
  1. safe_create_leading_directories(): on Windows, \ can separate path componentsMichael Haggerty, Jan 18, 2014
  2. Sebastian SchuberthJan 19, 2014
  3. Junio C HamanoJan 21, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.