{"thread":{"id":"24249","subject":"Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","startedAt":"2010-07-01T08:54:26Z","lastAt":"2010-07-01T10:36:02Z","messageCount":7,"participants":["Frank Terbeck","Michael J Gruber"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"144567","messageId":"8739w3fupp.fsf@ft.bewatermyfriend.org","threadId":"24249","inReplyTo":null,"subject":"Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","fromName":"Frank Terbeck","fromEmail":"ft@bewatermyfriend.org","sentAt":"2010-07-01T08:54:26Z","receivedAt":"2010-07-01T08:54:26Z","isPatch":false,"sender":{"key":"ft@bewatermyfriend.org","avatar":null},"body":"Hello,\n\nRunning the test suite currently breaks for me at this point:\n\n[snip]\n*   ok 16: info --url added-file\n* FAIL 17: info added-directory\n\n                mkdir gitwc/added-directory svnwc/added-directory &&\n                ptouch gitwc/added-directory svnwc/added-directory &&\n                touch gitwc/added-directory/.placeholder &&\n                cd svnwc &&\n                        svn_cmd add added-directory > /dev/null &&\n                cd .. &&\n                cd gitwc &&\n                        git add added-directory &&\n                cd .. &&\n                (cd svnwc; svn info added-directory)            >\n                expected.info-added-directory &&\n                (cd gitwc; git svn info\n                added-directory)                >\n                actual.info-added-directory &&\n                test_cmp expected.info-added-directory\n                actual.info-added-directory\n\n*   ok 18: info --url added-directory\n[...]\n* failed 1 among 38 test(s)\nmake[2]: *** [t9119-git-svn-info.sh] Error 1\n[snap]\n\nI briefly searched the list archives via gmane but didn't find this\nreported yet. I've got no idea what's going on in that test, as I'm no\nexpert in subversion matters at all. So I can't be of more help than\nsaying \"well, it breaks for me.\". :)\n\nAnyway, my svn version is \"svn, version 1.6.9 (r901367)\". The test suite\nI ran is from yesterdays git master branch. If it matters, this is a\nLinux box running a debian 2.6.26-2-686 kernel; the file system in\nquestion is `ext3' being mounted `noatime'.\n\nRegards, Frank\n\nPS: I hope I Cced the right group of people.\n\n-- \nIn protocol design, perfection has been reached not when there is\nnothing left to add, but when there is nothing left to take away.\n                                                  -- RFC 1925\n"},{"id":"144573","messageId":"4C2C5D2F.90000@drmicha.warpmail.net","threadId":"24249","inReplyTo":"8739w3fupp.fsf@ft.bewatermyfriend.org","subject":"Re: Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-07-01T09:17:35Z","receivedAt":"2010-07-01T09:17:35Z","isPatch":false,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Frank Terbeck venit, vidit, dixit 01.07.2010 10:54:\n> Hello,\n> \n> Running the test suite currently breaks for me at this point:\n> \n> [snip]\n> *   ok 16: info --url added-file\n> * FAIL 17: info added-directory\n> \n>                 mkdir gitwc/added-directory svnwc/added-directory &&\n>                 ptouch gitwc/added-directory svnwc/added-directory &&\n>                 touch gitwc/added-directory/.placeholder &&\n>                 cd svnwc &&\n>                         svn_cmd add added-directory > /dev/null &&\n>                 cd .. &&\n>                 cd gitwc &&\n>                         git add added-directory &&\n>                 cd .. &&\n>                 (cd svnwc; svn info added-directory)            >\n>                 expected.info-added-directory &&\n>                 (cd gitwc; git svn info\n>                 added-directory)                >\n>                 actual.info-added-directory &&\n>                 test_cmp expected.info-added-directory\n>                 actual.info-added-directory\n> \n> *   ok 18: info --url added-directory\n> [...]\n> * failed 1 among 38 test(s)\n> make[2]: *** [t9119-git-svn-info.sh] Error 1\n> [snap]\n> \n> I briefly searched the list archives via gmane but didn't find this\n> reported yet. I've got no idea what's going on in that test, as I'm no\n> expert in subversion matters at all. So I can't be of more help than\n> saying \"well, it breaks for me.\". :)\n> \n> Anyway, my svn version is \"svn, version 1.6.9 (r901367)\". The test suite\n> I ran is from yesterdays git master branch. If it matters, this is a\n> Linux box running a debian 2.6.26-2-686 kernel; the file system in\n> question is `ext3' being mounted `noatime'.\n\nIt works for me with the exact same svn version on Fedora 13.\n\nPlease run the test like that:\ncd t\n./t9119-git-svn-info.sh -i\n\nThis stop immediately after the failed test. Then, please tell us the\ncontents of the actual.. and expected... files in the trash... subdir.\n\n\nMichael\n\n> \n> Regards, Frank\n> \n> PS: I hope I Cced the right group of people.\n\nEric's the svn guy, the rest is just other helpful people ;)\n"},{"id":"144574","messageId":"87y6dveekn.fsf@ft.bewatermyfriend.org","threadId":"24249","inReplyTo":"4C2C5D2F.90000@drmicha.warpmail.net","subject":"Re: Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","fromName":"Frank Terbeck","fromEmail":"ft@bewatermyfriend.org","sentAt":"2010-07-01T09:28:24Z","receivedAt":"2010-07-01T09:28:24Z","isPatch":false,"sender":{"key":"ft@bewatermyfriend.org","avatar":null},"body":"Michael J Gruber wrote:\n> Frank Terbeck venit, vidit, dixit 01.07.2010 10:54:\n>> [snip]\n>> *   ok 16: info --url added-file\n>> * FAIL 17: info added-directory\n[...]\n>> *   ok 18: info --url added-directory\n>> [...]\n>> * failed 1 among 38 test(s)\n>> make[2]: *** [t9119-git-svn-info.sh] Error 1\n>> [snap]\n[...]\n>> Anyway, my svn version is \"svn, version 1.6.9 (r901367)\". The test suite\n>> I ran is from yesterdays git master branch. If it matters, this is a\n>> Linux box running a debian 2.6.26-2-686 kernel; the file system in\n>> question is `ext3' being mounted `noatime'.\n>\n> It works for me with the exact same svn version on Fedora 13.\n>\n> Please run the test like that:\n> cd t\n> ./t9119-git-svn-info.sh -i\n>\n> This stop immediately after the failed test. Then, please tell us the\n> contents of the actual.. and expected... files in the trash... subdir.\n\nOkay.\n\nexpected.info-added-directory:\n[snip]\nPath: added-directory\nURL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\nRepository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\nRepository UUID: 638d0b70-c7df-46bd-9c56-aff8052b4ab8\nRevision: 0\nNode Kind: directory\nSchedule: add\n[snap]\n\nactual.info-added-directory:\n[snip]\nPath: added-directory\nURL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\nRepository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\nRevision: 0\nNode Kind: directory\nSchedule: add\n[snap]\n\nAnd a unified diff of the two:\n[snip]\n--- trash directory.t9119-git-svn-info/actual.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n+++ trash directory.t9119-git-svn-info/expected.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n@@ -1,6 +1,7 @@\n Path: added-directory\n URL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\n Repository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\n+Repository UUID: 638d0b70-c7df-46bd-9c56-aff8052b4ab8\n Revision: 0\n Node Kind: directory\n Schedule: add\n[snap]\n\nRegards, Frank\n"},{"id":"144577","messageId":"4C2C647F.7090909@drmicha.warpmail.net","threadId":"24249","inReplyTo":"87y6dveekn.fsf@ft.bewatermyfriend.org","subject":"Re: Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-07-01T09:48:47Z","receivedAt":"2010-07-01T09:48:47Z","isPatch":false,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Frank Terbeck venit, vidit, dixit 01.07.2010 11:28:\n> Michael J Gruber wrote:\n>> Frank Terbeck venit, vidit, dixit 01.07.2010 10:54:\n>>> [snip]\n>>> *   ok 16: info --url added-file\n>>> * FAIL 17: info added-directory\n> [...]\n>>> *   ok 18: info --url added-directory\n>>> [...]\n>>> * failed 1 among 38 test(s)\n>>> make[2]: *** [t9119-git-svn-info.sh] Error 1\n>>> [snap]\n> [...]\n>>> Anyway, my svn version is \"svn, version 1.6.9 (r901367)\". The test suite\n>>> I ran is from yesterdays git master branch. If it matters, this is a\n>>> Linux box running a debian 2.6.26-2-686 kernel; the file system in\n>>> question is `ext3' being mounted `noatime'.\n>>\n>> It works for me with the exact same svn version on Fedora 13.\n>>\n>> Please run the test like that:\n>> cd t\n>> ./t9119-git-svn-info.sh -i\n>>\n>> This stop immediately after the failed test. Then, please tell us the\n>> contents of the actual.. and expected... files in the trash... subdir.\n> \n> Okay.\n> \n> expected.info-added-directory:\n> [snip]\n> Path: added-directory\n> URL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\n> Repository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\n> Repository UUID: 638d0b70-c7df-46bd-9c56-aff8052b4ab8\n> Revision: 0\n> Node Kind: directory\n> Schedule: add\n> [snap]\n> \n> actual.info-added-directory:\n> [snip]\n> Path: added-directory\n> URL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\n> Repository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\n> Revision: 0\n> Node Kind: directory\n> Schedule: add\n> [snap]\n> \n> And a unified diff of the two:\n> [snip]\n> --- trash directory.t9119-git-svn-info/actual.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n> +++ trash directory.t9119-git-svn-info/expected.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n> @@ -1,6 +1,7 @@\n>  Path: added-directory\n>  URL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\n>  Repository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\n> +Repository UUID: 638d0b70-c7df-46bd-9c56-aff8052b4ab8\n\n\"Nachtigall, ick hör Dir trapsen.\" [I really don't know how to translate\nthis...]\n\nAnyway, this is something I had fixed previously before realizing it\ndoesn't need a fix and something else is wrong:\n\nIt indicates that Git's svn-bindings (git-svn) use a different svn\nversion than the actual \"svn\". (There used to be a bug where Git was\nwrong about the version of the bindings but that is fixed.)\n\nDo the following agree for you:\n\ngit svn --version\nsvn --version\n\nDo you have any private copies of svn/svn bindings in your path or perl\npath?\n\n>  Revision: 0\n>  Node Kind: directory\n>  Schedule: add\n> [snap]\n> \n> Regards, Frank\n\nMichael\n"},{"id":"144581","messageId":"87pqz7ec79.fsf@ft.bewatermyfriend.org","threadId":"24249","inReplyTo":"4C2C647F.7090909@drmicha.warpmail.net","subject":"Re: Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","fromName":"Frank Terbeck","fromEmail":"ft@bewatermyfriend.org","sentAt":"2010-07-01T10:19:38Z","receivedAt":"2010-07-01T10:19:38Z","isPatch":false,"sender":{"key":"ft@bewatermyfriend.org","avatar":null},"body":"Michael J Gruber wrote:\n[...]\n>> And a unified diff of the two:\n>> [snip]\n>> --- trash directory.t9119-git-svn-info/actual.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n>> +++ trash directory.t9119-git-svn-info/expected.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n>> @@ -1,6 +1,7 @@\n>>  Path: added-directory\n>>  URL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\n>>  Repository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\n>> +Repository UUID: 638d0b70-c7df-46bd-9c56-aff8052b4ab8\n>\n> \"Nachtigall, ick hör Dir trapsen.\" [I really don't know how to translate\n> this...]\n\nNo need. :)\n\n> Anyway, this is something I had fixed previously before realizing it\n> doesn't need a fix and something else is wrong:\n>\n> It indicates that Git's svn-bindings (git-svn) use a different svn\n> version than the actual \"svn\". (There used to be a bug where Git was\n> wrong about the version of the bindings but that is fixed.)\n>\n> Do the following agree for you:\n>\n> git svn --version\n> svn --version\n>\n> Do you have any private copies of svn/svn bindings in your path or perl\n> path?\n\nOkay, it wasn't really a private copy, but you're otherwise\nspot-on. Here's what was wrong:\n\nI didn't realise, `git-svn' was using Perl's subversion bindings. This\nhappens on my laptop which runs debian stable (lenny). I was upgrading\nmy subversion package to the one provided by backports.org (a service\nthat provides backports of newer versions of certain software packages\nto debian stable). While that worked nicely, the subversion bindings for\nPerl are provided by a package called `libsvn-perl' which was kept at\nthe version in stable, which is 1.5.1. Backports.org also provides an\nupgrade for that package. Now both \"svn --version\" and \"git svn\n--version\" report the same subversion version and the test passes again.\n\n\nMaybe it would be good if git-svn or maybe just the test suite checked\nwhether subversion's and the Perl binding's version matched?\n\nRegards, Frank\n\n-- \nIn protocol design, perfection has been reached not when there is\nnothing left to add, but when there is nothing left to take away.\n                                                  -- RFC 1925\n"},{"id":"144582","messageId":"4C2C6D64.4060802@drmicha.warpmail.net","threadId":"24249","inReplyTo":"87pqz7ec79.fsf@ft.bewatermyfriend.org","subject":"Re: Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2010-07-01T10:26:44Z","receivedAt":"2010-07-01T10:26:44Z","isPatch":false,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Frank Terbeck venit, vidit, dixit 01.07.2010 12:19:\n> Michael J Gruber wrote:\n> [...]\n>>> And a unified diff of the two:\n>>> [snip]\n>>> --- trash directory.t9119-git-svn-info/actual.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n>>> +++ trash directory.t9119-git-svn-info/expected.info-added-directory 2010-07-01 11:22:16.000000000 +0200\n>>> @@ -1,6 +1,7 @@\n>>>  Path: added-directory\n>>>  URL: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo/added-directory\n>>>  Repository Root: file:///usr/src/sys/git/t/trash%20directory.t9119-git-svn-info/svnrepo\n>>> +Repository UUID: 638d0b70-c7df-46bd-9c56-aff8052b4ab8\n>>\n>> \"Nachtigall, ick hör Dir trapsen.\" [I really don't know how to translate\n>> this...]\n> \n> No need. :)\n> \n>> Anyway, this is something I had fixed previously before realizing it\n>> doesn't need a fix and something else is wrong:\n>>\n>> It indicates that Git's svn-bindings (git-svn) use a different svn\n>> version than the actual \"svn\". (There used to be a bug where Git was\n>> wrong about the version of the bindings but that is fixed.)\n>>\n>> Do the following agree for you:\n>>\n>> git svn --version\n>> svn --version\n>>\n>> Do you have any private copies of svn/svn bindings in your path or perl\n>> path?\n> \n> Okay, it wasn't really a private copy, but you're otherwise\n> spot-on. Here's what was wrong:\n\nTurns out you cc'ed the right persons :)\n\n> \n> I didn't realise, `git-svn' was using Perl's subversion bindings. This\n> happens on my laptop which runs debian stable (lenny). I was upgrading\n> my subversion package to the one provided by backports.org (a service\n> that provides backports of newer versions of certain software packages\n> to debian stable). While that worked nicely, the subversion bindings for\n> Perl are provided by a package called `libsvn-perl' which was kept at\n> the version in stable, which is 1.5.1. Backports.org also provides an\n> upgrade for that package. Now both \"svn --version\" and \"git svn\n> --version\" report the same subversion version and the test passes again.\n> \n> \n> Maybe it would be good if git-svn or maybe just the test suite checked\n> whether subversion's and the Perl binding's version matched?\n\nThen you would not be able to use git-svn with your setup!\n\nHaving svn and its bindings at different versions is perfectly fine for\ngit-svn: it uses and cares about the bindings only (and adjusts\naccording to the binding's version).\n\nIt's only the test suite which cares, because it tries to make sure that\n\"svn\" and \"git-svn\" behave as similar as possible.\n\nWe should probably check in the test suite before triggering false\nalarms, I'll look into that.\n\nCheers,\nMichael\n"},{"id":"144583","messageId":"87lj9vebfx.fsf@ft.bewatermyfriend.org","threadId":"24249","inReplyTo":"4C2C6D64.4060802@drmicha.warpmail.net","subject":"Re: Test 17 of `t9119-git-svn-info.sh' fails with svn 1.6.9","fromName":"Frank Terbeck","fromEmail":"ft@bewatermyfriend.org","sentAt":"2010-07-01T10:36:02Z","receivedAt":"2010-07-01T10:36:02Z","isPatch":false,"sender":{"key":"ft@bewatermyfriend.org","avatar":null},"body":"Michael J Gruber wrote:\n> Frank Terbeck venit, vidit, dixit 01.07.2010 12:19:\n>> Michael J Gruber wrote:\n[...]\n>>> Do the following agree for you:\n>>>\n>>> git svn --version\n>>> svn --version\n>>>\n>>> Do you have any private copies of svn/svn bindings in your path or perl\n>>> path?\n>> \n>> Okay, it wasn't really a private copy, but you're otherwise\n>> spot-on. Here's what was wrong:\n>\n> Turns out you cc'ed the right persons :)\n\nHeh, seems so. :)\n\n>> I didn't realise, `git-svn' was using Perl's subversion bindings. This\n>> happens on my laptop which runs debian stable (lenny). I was upgrading\n>> my subversion package to the one provided by backports.org (a service\n>> that provides backports of newer versions of certain software packages\n>> to debian stable). While that worked nicely, the subversion bindings for\n>> Perl are provided by a package called `libsvn-perl' which was kept at\n>> the version in stable, which is 1.5.1. Backports.org also provides an\n>> upgrade for that package. Now both \"svn --version\" and \"git svn\n>> --version\" report the same subversion version and the test passes again.\n>> \n>> \n>> Maybe it would be good if git-svn or maybe just the test suite checked\n>> whether subversion's and the Perl binding's version matched?\n>\n> Then you would not be able to use git-svn with your setup!\n\nWell, now it would. :)\nBut I get your point.\n\n> Having svn and its bindings at different versions is perfectly fine for\n> git-svn: it uses and cares about the bindings only (and adjusts\n> according to the binding's version).\n\nOkay.\n\n> It's only the test suite which cares, because it tries to make sure that\n> \"svn\" and \"git-svn\" behave as similar as possible.\n>\n> We should probably check in the test suite before triggering false\n> alarms, I'll look into that.\n\nCool thanks.\n\nAnd thank you for the quick response and solution.\n\nRegards, Frank\n"}]}