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

Re: [PATCH 2/3] Windows: a test_cmp that is agnostic to random LF <> CRLF conversions

From
Johannes Sixt <j6t@kdbg.org>
Date
Nov 2, 2013, 20:40 UTC
Message-ID
<52756332.50404@kdbg.org>
In-Reply-To
<52756198.2070900@gmail.com>
Am 02.11.2013 21:33, schrieb Sebastian Schuberth:
Show 12 quoted lines
> On 26.10.2013 21:17, Johannes Sixt wrote:
> 
>> In a number of tests, output that was produced by a shell script is
>> compared to expected output using test_cmp. Unfortunately, the MSYS
>> bash--
>> when invoked via git, such as in hooks--converts LF to CRLF on output
>> (as produced by echo and printf), which leads to many false positives.
> 
> As you correctly point out the LF vs. CRLF issues are caused by MSYS
> bash. Then why are the functions called mingw_* instead of msys_*? I'd
> prefer to make very clear that not MinGW but MSYS is the culprit
> concerning the LF vs. CRLF issues (and also the path mangling issues).

It's a historical accident. We already have the prerequisite MINGW to work around various annoyances, most of which are caused by MSYS bash. I didn't want to invent a new prefix.

-- Hannes
-- 
-- 
*** Please reply-to-all at all times ***
*** (do not pretend to know who is subscribed and who is not) ***
*** Please avoid top-posting. ***
The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.

You received this message because you are subscribed to the Google
Groups "msysGit" group.
To post to this group, send email to msysgit@googlegroups.com
To unsubscribe from this group, send email to
msysgit+unsubscribe@googlegroups.com
For more options, and view previous threads, visit this group at
http://groups.google.com/group/msysgit?hl=en_US?hl=en

--- 
You received this message because you are subscribed to the Google Groups "msysGit" group.
To unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
Previous: Sebastian SchuberthNext: Sebastian Schuberth
Message 5 of 8 in “Fix tests on Windows caused by random CRLF output”
  1. 0/3 Fix tests on Windows caused by random CRLF outputJohannes Sixt, Oct 26, 2013
  2. 1/3 t5300-pack-object: do not compare binary data using test_cmpJohannes Sixt, Oct 26, 2013
  3. 2/3 Windows: a test_cmp that is agnostic to random LF <> CRLF conversionsJohannes Sixt, Oct 26, 2013
  4. Sebastian SchuberthNov 2, 2013
  5. Johannes SixtNov 2, 2013
  6. Sebastian SchuberthNov 2, 2013
  7. Johannes SixtNov 3, 2013
  8. 3/3 tests: undo special treatment of CRLF for WindowsJohannes Sixt, Oct 26, 2013

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.