{"thread":{"id":"30757","subject":"[PATCH] t0020: make sure the test file is lf to start with","startedAt":"2012-06-10T10:32:32Z","lastAt":"2012-06-10T12:50:35Z","messageCount":4,"participants":["vfr@lyx.org","Johannes Schindelin","Vincent van Ravesteijn"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"193241","messageId":"1339324352-7376-1-git-send-email-vfr@lyx.org","threadId":"30757","inReplyTo":null,"subject":"[PATCH] t0020: make sure the test file is lf to start with","fromName":"","fromEmail":"vfr@lyx.org","sentAt":"2012-06-10T10:32:32Z","receivedAt":"2012-06-10T10:32:32Z","isPatch":true,"sender":{"key":"vfr@lyx.org","avatar":"https://avatars.githubusercontent.com/u/687868?v=4"},"body":"From: Vincent van Ravesteijn <vfr@lyx.org>\n\nOn Windows, the perl interpreter used by q_to_nul also converts the\neol-style to crlf. Because the eol-style is now wrong from the beginning, the\nlater tests '.gitattributes says two is binary' and 'checkout with\nexisting .gitattributes' fail. This patch makes sure that the eol-style is\nlf when the test file is setup and makes the tests to pass also on\nWindows.\n\nSigned-off-by: Vincent van Ravesteijn <vfr@lyx.org>\n\n---\nI'm not sure everyone will like this solution to fix a problem that is only present on Windows. I'd be happy to discuss a different solution. \n\n t/t0020-crlf.sh |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/t/t0020-crlf.sh b/t/t0020-crlf.sh\nindex 1a8f44c..4c41b6e 100755\n--- a/t/t0020-crlf.sh\n+++ b/t/t0020-crlf.sh\n@@ -15,7 +15,7 @@ test_expect_success setup '\n \tfor w in Hello world how are you; do echo $w; done >one &&\n \tmkdir dir &&\n \tfor w in I am very very fine thank you; do echo $w; done >dir/two &&\n-\tfor w in Oh here is NULQin text here; do echo $w; done | q_to_nul >three &&\n+\tfor w in Oh here is NULQin text here; do echo $w; done | q_to_nul | remove_cr >three &&\n \tgit add . &&\n \n \tgit commit -m initial &&\n-- \n1.7.9.msysgit.0\n"},{"id":"193243","messageId":"alpine.DEB.1.00.1206101145550.16012@bonsai2","threadId":"30757","inReplyTo":"1339324352-7376-1-git-send-email-vfr@lyx.org","subject":"Re: [msysGit] [PATCH] t0020: make sure the test file is lf to start with","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2012-06-10T10:47:46Z","receivedAt":"2012-06-10T10:47:46Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Vincent,\n\nOn Sun, 10 Jun 2012, vfr@lyx.org wrote:\n\n> On Windows, the perl interpreter used by q_to_nul also converts the\n> eol-style to crlf.\n\nFirst: thanks for working on this, and thanks also for Cc:ing the msysgit\nlist.\n\n> Because the eol-style is now wrong from the beginning, the later tests\n> '.gitattributes says two is binary' and 'checkout with existing\n> .gitattributes' fail.\n\nI wonder why it does not fail here:\n\n\thttps://qa.nest-initiative.org/job/msysgit-test/24/console\n\nIOW I think there is a difference between the msysGit setup and your\nsetup...\n\nCiao,\nJohannes\n"},{"id":"193244","messageId":"4FD47E99.2000004@lyx.org","threadId":"30757","inReplyTo":"alpine.DEB.1.00.1206101145550.16012@bonsai2","subject":"Re: [PATCH] t0020: make sure the test file is lf to start with","fromName":"Vincent van Ravesteijn","fromEmail":"vfr@lyx.org","sentAt":"2012-06-10T11:01:45Z","receivedAt":"2012-06-10T11:01:45Z","isPatch":true,"sender":{"key":"vfr@lyx.org","avatar":"https://avatars.githubusercontent.com/u/687868?v=4"},"body":"Op 10-6-2012 12:47, Johannes Schindelin schreef:\n> Hi Vincent,\n>\n>> Because the eol-style is now wrong from the beginning, the later tests\n>> '.gitattributes says two is binary' and 'checkout with existing\n>> .gitattributes' fail.\n> I wonder why it does not fail here:\n>\n> \thttps://qa.nest-initiative.org/job/msysgit-test/24/console\n>\n> IOW I think there is a difference between the msysGit setup and your\n> setup...\n\nI compiled git using MSVC and try to run it in the Windows cmd prompt. \nConsequently it doesn't use the perl that is distributed with msysgit, \nbut it uses the perl that I've installed on my Windows system \n(Strawberry Perl; http://strawberryperl.com/).\n\nWhen I modify the paths such that git uses the perl.exe distributed with \nmsysgit the tests run ok.\n\nVincent\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"},{"id":"193256","messageId":"alpine.DEB.1.00.1206101349180.16012@bonsai2","threadId":"30757","inReplyTo":"4FD47E99.2000004@lyx.org","subject":"Re: [PATCH] t0020: make sure the test file is lf to start with","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2012-06-10T12:50:35Z","receivedAt":"2012-06-10T12:50:35Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Vincent,\n\nOn Sun, 10 Jun 2012, Vincent van Ravesteijn wrote:\n\n> Op 10-6-2012 12:47, Johannes Schindelin schreef:\n> >\n> > > Because the eol-style is now wrong from the beginning, the later\n> > > tests '.gitattributes says two is binary' and 'checkout with\n> > > existing .gitattributes' fail.\n>\n> > I wonder why it does not fail here:\n> >\n> >  https://qa.nest-initiative.org/job/msysgit-test/24/console\n> >\n> > IOW I think there is a difference between the msysGit setup and your\n> > setup...\n> \n> I compiled git using MSVC and try to run it in the Windows cmd prompt.\n> Consequently it doesn't use the perl that is distributed with msysgit,\n> but it uses the perl that I've installed on my Windows system\n> (Strawberry Perl; http://strawberryperl.com/).\n\nAh, now it makes sense. Maybe a valuable information to convey in the\ncommit message?\n\n> When I modify the paths such that git uses the perl.exe distributed with\n> msysgit the tests run ok.\n\nAgain, a valuable information for the commit message.\n\nAlso, I suspect that many, many more places should be affected by that\nPerl's behavior...\n\nCiao,\nJohannes\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"}]}