{"thread":{"id":"35210","subject":"[PATCH 2/3] Windows: a test_cmp that is agnostic to random LF <> CRLF conversions","startedAt":"2013-10-26T19:17:13Z","lastAt":"2013-11-03T08:32:00Z","messageCount":8,"participants":["Johannes Sixt","Sebastian Schuberth"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"229575","messageId":"cover.1382814437.git.j6t@kdbg.org","threadId":"35210","inReplyTo":null,"subject":"[PATCH 0/3] Fix tests on Windows caused by random CRLF output","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2013-10-26T19:17:13Z","receivedAt":"2013-10-26T19:17:13Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"When git, the executable, invokes a bash script (hooks, scripted\ngit commands, ...), bash writes its output with CRLF. This causes\ntests to fail because the expected output does not contain CR, only LF.\n\nThis fixes the tests by replacing test_cmp with a diff computation\nthat disregards the CR.\n\nPatch 1 replaces test_cmp with cmp when binary data is compared.\nPatch 2 is the actual fix.\nPatch 3 removes some work-arounds that are not necessary anymore.\n\nJohannes Sixt (3):\n  t5300-pack-object: do not compare binary data using test_cmp\n  Windows: a test_cmp that is agnostic to random LF <> CRLF conversions\n  tests: undo special treatment of CRLF for Windows\n\n t/lib-credential.sh          |  4 ---\n t/t5300-pack-object.sh       | 10 +++----\n t/t7407-submodule-foreach.sh |  4 ---\n t/t9001-send-email.sh        |  1 -\n t/test-lib-functions.sh      | 66 ++++++++++++++++++++++++++++++++++++++++++++\n t/test-lib.sh                |  1 +\n 6 files changed, 72 insertions(+), 14 deletions(-)\n\n-- \n1.8.4.33.gd68f7e8\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"},{"id":"229574","messageId":"f950a2a60c0798650646157b1785f8e9e558b7af.1382814437.git.j6t@kdbg.org","threadId":"35210","inReplyTo":"cover.1382814437.git.j6t@kdbg.org","subject":"[PATCH 1/3] t5300-pack-object: do not compare binary data using test_cmp","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2013-10-26T19:17:14Z","receivedAt":"2013-10-26T19:17:14Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Users may set test_cmp to a comparison tool of their liking. The intent is\nthat the tool performs comparison of line-oriented texts. However, t5300\nuses it also to compare binary data. Change those tests to use 'cmp'.\n\nSigned-off-by: Johannes Sixt <j6t@kdbg.org>\n---\n t/t5300-pack-object.sh | 10 +++++-----\n 1 file changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/t/t5300-pack-object.sh b/t/t5300-pack-object.sh\nindex a07c871..61e787d 100755\n--- a/t/t5300-pack-object.sh\n+++ b/t/t5300-pack-object.sh\n@@ -151,7 +151,7 @@ test_expect_success \\\n \t    git cat-file $t $object || return 1\n \t done <obj-list\n     } >current &&\n-    test_cmp expect current'\n+    cmp expect current'\n \n test_expect_success \\\n     'use packed deltified (REF_DELTA) objects' \\\n@@ -166,7 +166,7 @@ test_expect_success \\\n \t    git cat-file $t $object || return 1\n \t done <obj-list\n     } >current &&\n-    test_cmp expect current'\n+    cmp expect current'\n \n test_expect_success \\\n     'use packed deltified (OFS_DELTA) objects' \\\n@@ -181,7 +181,7 @@ test_expect_success \\\n \t    git cat-file $t $object || return 1\n \t done <obj-list\n     } >current &&\n-    test_cmp expect current'\n+    cmp expect current'\n \n unset GIT_OBJECT_DIRECTORY\n \n@@ -195,9 +195,9 @@ test_expect_success 'survive missing objects/pack directory' '\n \t\trm -fr $GOP &&\n \t\tgit index-pack --stdin --keep=test <../test-3-${packname_3}.pack &&\n \t\ttest -f $GOP/pack-${packname_3}.pack &&\n-\t\ttest_cmp $GOP/pack-${packname_3}.pack ../test-3-${packname_3}.pack &&\n+\t\tcmp $GOP/pack-${packname_3}.pack ../test-3-${packname_3}.pack &&\n \t\ttest -f $GOP/pack-${packname_3}.idx &&\n-\t\ttest_cmp $GOP/pack-${packname_3}.idx ../test-3-${packname_3}.idx &&\n+\t\tcmp $GOP/pack-${packname_3}.idx ../test-3-${packname_3}.idx &&\n \t\ttest -f $GOP/pack-${packname_3}.keep\n \t)\n '\n-- \n1.8.4.33.gd68f7e8\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"},{"id":"229573","messageId":"e64878fec3f026802e8d3958a1e6213428cab778.1382814437.git.j6t@kdbg.org","threadId":"35210","inReplyTo":"cover.1382814437.git.j6t@kdbg.org","subject":"[PATCH 2/3] Windows: a test_cmp that is agnostic to random LF <> CRLF conversions","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2013-10-26T19:17:15Z","receivedAt":"2013-10-26T19:17:15Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"In a number of tests, output that was produced by a shell script is\ncompared to expected output using test_cmp. Unfortunately, the MSYS bash--\nwhen invoked via git, such as in hooks--converts LF to CRLF on output\n(as produced by echo and printf), which leads to many false positives.\n\nImplements a diff tool that undoes the converted CRLF. To avoid that\nsub-processes are spawned (which is very slow on Windows), the tool is\nimplemented as a shell function. Diff is invoked as usual only when a\ndifference is detected by the shell code.\n\nSigned-off-by: Johannes Sixt <j6t@kdbg.org>\n---\n t/test-lib-functions.sh | 66 +++++++++++++++++++++++++++++++++++++++++++++++++\n t/test-lib.sh           |  1 +\n 2 files changed, 67 insertions(+)\n\ndiff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh\nindex a7e9aac..f987694 100644\n--- a/t/test-lib-functions.sh\n+++ b/t/test-lib-functions.sh\n@@ -710,3 +710,69 @@ test_ln_s_add () {\n \t\tgit update-index --add --cacheinfo 120000 $ln_s_obj \"$2\"\n \tfi\n }\n+\n+# The following mingw_* functions obey POSIX shell syntax, but are actually\n+# bash scripts, and are meant to be used only with bash on Windows.\n+\n+# A test_cmp function that treats LF and CRLF equal and avoids to fork\n+# diff when possible.\n+mingw_test_cmp () {\n+\t# Read text into shell variables and compare them. If the results\n+\t# are different, use regular diff to report the difference.\n+\tlocal test_cmp_a= test_cmp_b=\n+\n+\t# When text came from stdin (one argument is '-') we must feed it\n+\t# to diff.\n+\tlocal stdin_for_diff=\n+\n+\t# Since it is difficult to detect the difference between an\n+\t# empty input file and a failure to read the files, we go straight\n+\t# to diff if one of the inputs is empty.\n+\tif test -s \"$1\" && test -s \"$2\"\n+\tthen\n+\t\t# regular case: both files non-empty\n+\t\tmingw_read_file_strip_cr_ test_cmp_a <\"$1\"\n+\t\tmingw_read_file_strip_cr_ test_cmp_b <\"$2\"\n+\telif test -s \"$1\" && test \"$2\" = -\n+\tthen\n+\t\t# read 2nd file from stdin\n+\t\tmingw_read_file_strip_cr_ test_cmp_a <\"$1\"\n+\t\tmingw_read_file_strip_cr_ test_cmp_b\n+\t\tstdin_for_diff='<<<\"$test_cmp_b\"'\n+\telif test \"$1\" = - && test -s \"$2\"\n+\tthen\n+\t\t# read 1st file from stdin\n+\t\tmingw_read_file_strip_cr_ test_cmp_a\n+\t\tmingw_read_file_strip_cr_ test_cmp_b <\"$2\"\n+\t\tstdin_for_diff='<<<\"$test_cmp_a\"'\n+\tfi\n+\ttest -n \"$test_cmp_a\" &&\n+\ttest -n \"$test_cmp_b\" &&\n+\ttest \"$test_cmp_a\" = \"$test_cmp_b\" ||\n+\teval \"diff -u \\\"\\$@\\\" $stdin_for_diff\"\n+}\n+\n+# $1 is the name of the shell variable to fill in\n+mingw_read_file_strip_cr_ () {\n+\t# Read line-wise using LF as the line separator\n+\t# and use IFS to strip CR.\n+\tlocal line\n+\twhile :\n+\tdo\n+\t\tif IFS=$'\\r' read -r -d $'\\n' line\n+\t\tthen\n+\t\t\t# good\n+\t\t\tline=$line$'\\n'\n+\t\telse\n+\t\t\t# we get here at EOF, but also if the last line\n+\t\t\t# was not terminated by LF; in the latter case,\n+\t\t\t# some text was read\n+\t\t\tif test -z \"$line\"\n+\t\t\tthen\n+\t\t\t\t# EOF, really\n+\t\t\t\tbreak\n+\t\t\tfi\n+\t\tfi\n+\t\teval \"$1=\\$$1\\$line\"\n+\tdone\n+}\ndiff --git a/t/test-lib.sh b/t/test-lib.sh\nindex 0fa7dfd..77e487b 100644\n--- a/t/test-lib.sh\n+++ b/t/test-lib.sh\n@@ -817,6 +817,7 @@ case $(uname -s) in\n \ttest_set_prereq NOT_CYGWIN\n \ttest_set_prereq SED_STRIPS_CR\n \ttest_set_prereq GREP_STRIPS_CR\n+\tGIT_TEST_CMP=mingw_test_cmp\n \t;;\n *CYGWIN*)\n \ttest_set_prereq POSIXPERM\n-- \n1.8.4.33.gd68f7e8\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"},{"id":"229576","messageId":"0ad7f072a5ded97057660882e5a7a56649488a41.1382814437.git.j6t@kdbg.org","threadId":"35210","inReplyTo":"cover.1382814437.git.j6t@kdbg.org","subject":"[PATCH 3/3] tests: undo special treatment of CRLF for Windows","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2013-10-26T19:17:16Z","receivedAt":"2013-10-26T19:17:16Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Signed-off-by: Johannes Sixt <j6t@kdbg.org>\n---\n t/lib-credential.sh          | 4 ----\n t/t7407-submodule-foreach.sh | 4 ----\n t/t9001-send-email.sh        | 1 -\n 3 files changed, 9 deletions(-)\n\ndiff --git a/t/lib-credential.sh b/t/lib-credential.sh\nindex 3c43ff1..957ae93 100755\n--- a/t/lib-credential.sh\n+++ b/t/lib-credential.sh\n@@ -18,10 +18,6 @@ check() {\n \t\tcat stderr &&\n \t\tfalse\n \tfi &&\n-\tif test_have_prereq MINGW\n-\tthen\n-\t\tdos2unix -q stderr\n-\tfi &&\n \ttest_cmp expect-stdout stdout &&\n \ttest_cmp expect-stderr stderr\n }\ndiff --git a/t/t7407-submodule-foreach.sh b/t/t7407-submodule-foreach.sh\nindex be93f10..b64c9ed 100755\n--- a/t/t7407-submodule-foreach.sh\n+++ b/t/t7407-submodule-foreach.sh\n@@ -254,10 +254,6 @@ test_expect_success 'ensure \"status --cached --recursive\" preserves the --cached\n \t\t) &&\n \t\tgit submodule status --cached --recursive -- nested1 > ../actual\n \t) &&\n-\tif test_have_prereq MINGW\n-\tthen\n-\t\tdos2unix actual\n-\tfi &&\n \ttest_cmp expect actual\n '\n \ndiff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh\nindex 2813aa9..3119c8c 100755\n--- a/t/t9001-send-email.sh\n+++ b/t/t9001-send-email.sh\n@@ -23,7 +23,6 @@ test_expect_success $PREREQ \\\n       echo do\n       echo \"  echo \\\"!\\$a!\\\"\"\n       echo \"done >commandline\\$output\"\n-      test_have_prereq MINGW && echo \"dos2unix commandline\\$output\"\n       echo \"cat > msgtxt\\$output\"\n       ) >fake.sendmail &&\n      chmod +x ./fake.sendmail &&\n-- \n1.8.4.33.gd68f7e8\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"},{"id":"230042","messageId":"52756198.2070900@gmail.com","threadId":"35210","inReplyTo":"e64878fec3f026802e8d3958a1e6213428cab778.1382814437.git.j6t@kdbg.org","subject":"Re: [PATCH 2/3] Windows: a test_cmp that is agnostic to random LF <> CRLF conversions","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2013-11-02T20:33:28Z","receivedAt":"2013-11-02T20:33:28Z","isPatch":true,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On 26.10.2013 21:17, Johannes Sixt wrote:\n\n> In a number of tests, output that was produced by a shell script is\n> compared to expected output using test_cmp. Unfortunately, the MSYS bash--\n> when invoked via git, such as in hooks--converts LF to CRLF on output\n> (as produced by echo and printf), which leads to many false positives.\n\nAs you correctly point out the LF vs. CRLF issues are caused by MSYS \nbash. Then why are the functions called mingw_* instead of msys_*? I'd \nprefer to make very clear that not MinGW but MSYS is the culprit \nconcerning the LF vs. CRLF issues (and also the path mangling issues).\n\n-- \nSebastian Schuberth\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"},{"id":"230043","messageId":"52756332.50404@kdbg.org","threadId":"35210","inReplyTo":"52756198.2070900@gmail.com","subject":"Re: [PATCH 2/3] Windows: a test_cmp that is agnostic to random LF <> CRLF conversions","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2013-11-02T20:40:18Z","receivedAt":"2013-11-02T20:40:18Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Am 02.11.2013 21:33, schrieb Sebastian Schuberth:\n> On 26.10.2013 21:17, Johannes Sixt wrote:\n> \n>> In a number of tests, output that was produced by a shell script is\n>> compared to expected output using test_cmp. Unfortunately, the MSYS\n>> bash--\n>> when invoked via git, such as in hooks--converts LF to CRLF on output\n>> (as produced by echo and printf), which leads to many false positives.\n> \n> As you correctly point out the LF vs. CRLF issues are caused by MSYS\n> bash. Then why are the functions called mingw_* instead of msys_*? I'd\n> prefer to make very clear that not MinGW but MSYS is the culprit\n> concerning the LF vs. CRLF issues (and also the path mangling issues).\n\nIt's a historical accident. We already have the prerequisite MINGW to\nwork around various annoyances, most of which are caused by MSYS bash. I\ndidn't want to invent a new prefix.\n\n-- Hannes\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"},{"id":"230044","messageId":"CAHGBnuOHHAZUD_1WWu+WrAYKB1f0BonztbAFzVqhxP0bjAkyfw@mail.gmail.com","threadId":"35210","inReplyTo":"52756332.50404@kdbg.org","subject":"Re: [PATCH 2/3] Windows: a test_cmp that is agnostic to random LF <> CRLF conversions","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2013-11-02T20:47:35Z","receivedAt":"2013-11-02T20:47:35Z","isPatch":true,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Sat, Nov 2, 2013 at 9:40 PM, Johannes Sixt <j6t@kdbg.org> wrote:\n\n>>> In a number of tests, output that was produced by a shell script is\n>>> compared to expected output using test_cmp. Unfortunately, the MSYS\n>>> bash--\n>>> when invoked via git, such as in hooks--converts LF to CRLF on output\n>>> (as produced by echo and printf), which leads to many false positives.\n>>\n>> As you correctly point out the LF vs. CRLF issues are caused by MSYS\n>> bash. Then why are the functions called mingw_* instead of msys_*? I'd\n>> prefer to make very clear that not MinGW but MSYS is the culprit\n>> concerning the LF vs. CRLF issues (and also the path mangling issues).\n>\n> It's a historical accident. We already have the prerequisite MINGW to\n> work around various annoyances, most of which are caused by MSYS bash. I\n> didn't want to invent a new prefix.\n\nSo maybe it's a good point now to also change the MINGW prerequisite\nname to MSYS as part of your patch, and then name the functions more\nappropriately?\n\n-- \nSebastian Schuberth\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"},{"id":"230049","messageId":"52760A00.4080208@kdbg.org","threadId":"35210","inReplyTo":"CAHGBnuOHHAZUD_1WWu+WrAYKB1f0BonztbAFzVqhxP0bjAkyfw@mail.gmail.com","subject":"Re: [PATCH 2/3] Windows: a test_cmp that is agnostic to random LF <> CRLF conversions","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2013-11-03T08:32:00Z","receivedAt":"2013-11-03T08:32:00Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Am 02.11.2013 21:47, schrieb Sebastian Schuberth:\n> So maybe it's a good point now to also change the MINGW prerequisite\n> name to MSYS as part of your patch, and then name the functions more\n> appropriately?\n\nIt's too late. The series is already in master, and I see no point in a\nfollow-up patch of this kind as long as there is no additional benefit.\n\n-- Hannes\n\n-- \n-- \n*** Please reply-to-all at all times ***\n*** (do not pretend to know who is subscribed and who is not) ***\n*** Please avoid top-posting. ***\nThe msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.\n\nYou received this message because you are subscribed to the Google\nGroups \"msysGit\" group.\nTo post to this group, send email to msysgit@googlegroups.com\nTo unsubscribe from this group, send email to\nmsysgit+unsubscribe@googlegroups.com\nFor more options, and view previous threads, visit this group at\nhttp://groups.google.com/group/msysgit?hl=en_US?hl=en\n\n--- \nYou received this message because you are subscribed to the Google Groups \"msysGit\" group.\nTo unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.\nFor more options, visit https://groups.google.com/groups/opt_out.\n"}]}