{"thread":{"id":"785","subject":"cogito testsuite failure","startedAt":"2005-06-01T03:47:52Z","lastAt":"2005-06-01T05:55:10Z","messageCount":4,"participants":["Pavel Roskin","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"4339","messageId":"1117597672.13776.17.camel@dv","threadId":"785","inReplyTo":null,"subject":"cogito testsuite failure","fromName":"Pavel Roskin","fromEmail":"proski@gnu.org","sentAt":"2005-06-01T03:47:52Z","receivedAt":"2005-06-01T03:47:52Z","isPatch":false,"sender":{"key":"proski@gnu.org","avatar":null},"body":"Hello!\n\n\"make test\" fails for current cogito:\n\n*** t4008-diff-break-rewrite.sh ***\n* ok 1: setup\n* ok 2: change file1 with copy-edit of file0 and remove file0\n* ok 3: run diff with -B\n* ok 4: validate result of -B (#1)\n* ok 5: run diff with -B and -M\n* ok 6: validate result of -B -M (#2)\n* ok 7: swap file0 and file1\n* ok 8: run diff with -B\n* ok 9: validate result of -B (#3)\n* ok 10: run diff with -B and -M\n* ok 11: validate result of -B -M (#4)\n* ok 12: make file0 into something completely different\n* ok 13: run diff with -B\n* FAIL 14: validate result of -B (#5) compare_diff_raw current expected\n* ok 15: run diff with -B\n* FAIL 16: validate result of -B -M (#6) compare_diff_raw current expected\n* ok 17: run diff with -M\n* FAIL 18: validate result of -M (#7) compare_diff_raw current expected\n* ok 19: file1 edited to look like file0 and file0 rename-edited to file2\n* ok 20: run diff with -B\n* ok 21: validate result of -B (#8)\n* ok 22: run diff with -B -M\n* ok 23: validate result of -B -M (#9)\n* failed 3 among 23 test(s)\n\nThe failures happens because the test uses file ../README, which was\nchanged recently (and therefore, its checksum changed as well). Also,\nthis and several other tests refer to ../COPYING, which is going to\nchange as well (it has an old mailing address of FSF).\n\nI suggest that we use separate test files. It would be great if those\nfiles included text in several languages to test support for non-ASCII\nsymbols (maybe HELLO from emacs sources).\n\nOr maybe we could use short files embedded in the scripts. Or maybe we\ncould use sed to take significant portions of GPL that don't include\nLinus' foreword, any mailing addresses or \"19\" when it's used as two\nfirst digits of a year.\n\n-- \nRegards,\nPavel Roskin\n\n"},{"id":"4343","messageId":"7vzmuasnui.fsf@assigned-by-dhcp.cox.net","threadId":"785","inReplyTo":"1117597672.13776.17.camel@dv","subject":"Re: cogito testsuite failure","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-06-01T04:27:49Z","receivedAt":"2005-06-01T04:27:49Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"My intention was not to check the actual SHA1 in anything other\nthan t0000-basic.sh, but some test in t4000 series were buggy\nand ended up comparing the SHA1.\n\nI have since corrected them and I believe the test suite in the\ncurrent tip of Linus tree has the corrected compare_diff_raw and\nfriends in t/diff-lib.sh, which t4000 series use to avoid this\nproblem. I just hand munged COPYING and README and ran t4008 in\nthe Linus tree, and do not see the problem you are reporting.\n\nCould you verify that your test failure is coming from the\nversion that was before t/diff-lib.sh was introduced? Otherwise\nI have to investigate this further.\n\n\n"},{"id":"4345","messageId":"1117604049.5755.3.camel@dv","threadId":"785","inReplyTo":"7vzmuasnui.fsf@assigned-by-dhcp.cox.net","subject":"Re: cogito testsuite failure","fromName":"Pavel Roskin","fromEmail":"proski@gnu.org","sentAt":"2005-06-01T05:34:09Z","receivedAt":"2005-06-01T05:34:09Z","isPatch":false,"sender":{"key":"proski@gnu.org","avatar":null},"body":"Hi, Junio!\n\nOn Tue, 2005-05-31 at 21:27 -0700, Junio C Hamano wrote:\n> My intention was not to check the actual SHA1 in anything other\n> than t0000-basic.sh, but some test in t4000 series were buggy\n> and ended up comparing the SHA1.\n> \n> I have since corrected them and I believe the test suite in the\n> current tip of Linus tree has the corrected compare_diff_raw and\n> friends in t/diff-lib.sh, which t4000 series use to avoid this\n> problem. I just hand munged COPYING and README and ran t4008 in\n> the Linus tree, and do not see the problem you are reporting.\n> \n> Could you verify that your test failure is coming from the\n> version that was before t/diff-lib.sh was introduced? Otherwise\n> I have to investigate this further.\n\nThe failure in on the \"cogito\" branch\n(rsync://rsync.kernel.org/pub/scm/cogito/cogito.git), which doesn't have\nt/diff-lib.sh.\n\nThe \"git\" branch (rsync://www.kernel.org/pub/scm/git/git.git) is fine.\nSorry for the noise. I should have checked that branch too.\n\n-- \nRegards,\nPavel Roskin\n\n"},{"id":"4346","messageId":"7v1x7msjsx.fsf@assigned-by-dhcp.cox.net","threadId":"785","inReplyTo":"1117604049.5755.3.camel@dv","subject":"Re: cogito testsuite failure","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-06-01T05:55:10Z","receivedAt":"2005-06-01T05:55:10Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":">>>>> \"PR\" == Pavel Roskin writes:\n\nPR> The failure in on the \"cogito\" branch\nPR> (rsync://rsync.kernel.org/pub/scm/cogito/cogito.git), which doesn't have\nPR> t/diff-lib.sh.\n\nPR> The \"git\" branch (rsync://www.kernel.org/pub/scm/git/git.git) is fine.\nPR> Sorry for the noise. I should have checked that branch too.\n\nNo problem. Thanks.\n\n"}]}