{"thread":{"id":"11345","subject":"git-svn fix for broken symlinks","startedAt":"2007-12-18T16:11:56Z","lastAt":"2007-12-19T09:37:37Z","messageCount":3,"participants":["Sverre Johansen","Eric Wong","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"63632","messageId":"a74c0d680712180811m216699a4h7123039c3158c2ed@mail.gmail.com","threadId":"11345","inReplyTo":null,"subject":"git-svn fix for broken symlinks","fromName":"Sverre Johansen","fromEmail":"sverre.johansen@gmail.com","sentAt":"2007-12-18T16:11:56Z","receivedAt":"2007-12-18T16:11:56Z","isPatch":false,"sender":{"key":"sverre.johansen@gmail.com","avatar":null},"body":"Hi,\n\nThere is a bug in older versions of SVN that makes it possible to create\nsymlinks where the target is not the path to the file, but instead the\ncontent of the target file. The bug is described here [0].\n\nThis breaks git-svn because it expects files which is marked as symlinks\nto have the content \"link: \".\n\nThere was a thread about this back in July, resulting in this patch [1]. Is that\npatch planned to be merged anytime soon? I'm using it, and it works great\nfor me.\n\n[0]: http://subversion.tigris.org/issues/show_bug.cgi?id=2692\n[1]: http://kerneltrap.org/mailarchive/git/2007/7/19/252025\n-- \n Sverre Johansen\n"},{"id":"63726","messageId":"20071219084853.GB17787@muzzle","threadId":"11345","inReplyTo":"a74c0d680712180811m216699a4h7123039c3158c2ed@mail.gmail.com","subject":"Re: git-svn fix for broken symlinks","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2007-12-19T08:48:53Z","receivedAt":"2007-12-19T08:48:53Z","isPatch":false,"sender":{"key":"e@80x24.org","avatar":null},"body":"Sverre Johansen wrote:\n> Hi,\n> \n> There is a bug in older versions of SVN that makes it possible to create\n> symlinks where the target is not the path to the file, but instead the\n> content of the target file. The bug is described here [0].\n> \n> This breaks git-svn because it expects files which is marked as symlinks\n> to have the content \"link: \".\n> \n> There was a thread about this back in July, resulting in this patch [1]. Is that\n> patch planned to be merged anytime soon? I'm using it, and it works great\n> for me.\n> \n> [0]: http://subversion.tigris.org/issues/show_bug.cgi?id=2692\n> [1]: http://kerneltrap.org/mailarchive/git/2007/7/19/252025\n\nSverre: Thanks for the reminder, I'm quite absent-minded :/\n\nI've pushed out the patch to http://git.bogomips.org/git-svn.git\n(or git://, if you prefer) along with two others\n\nJunio: please pull, thanks.\n\ncommit 540424b263d626aa762757a31ca9b4becc7fb513\nAuthor: Eric Wong \nDate: Wed Dec 19 00:31:43 2007 -0800\n\n git-svn: avoid warning when run without arguments\n \n While we're in the area, finish writing a halfway-written\n comment describing what that block does...\n \n Signed-off-by: Eric Wong \n\ncommit 7fc35e0e94782bbbefb920875813519038659930\nAuthor: Eric Wong \nDate: Wed Dec 19 00:06:45 2007 -0800\n\n git-svn: workaround a for broken symlinks in SVN\n \n It's possible for bad clients to commit symlinks without the\n 5-character \"link \" prefix in symlinks. So guard around this\n bug in SVN and make a best effort to create symlinks if the\n \"link \" prefix is missing.\n \n More information on this SVN bug is described here:\n http://subversion.tigris.org/issues/show_bug.cgi?id=2692\n \n To be pedantic, there is still a corner case that neither we nor\n SVN can handle: If somebody made a link using a broken SVN\n client where \"link \" is the first part of its path, e.g.\n \"link sausage\", then we'd end up having a symlink which points\n to \"sausage\" because we incorrectly stripped the \"link \".\n Hopefully this hasn't happened in practice, but if it has,\n it's not our fault SVN is broken :)\n \n Thanks to Benoit Sigoure and Sverre Johansen for reporting\n and feedback.\n \n Signed-off-by: Eric Wong \n\ncommit ad94802a7bc599ade73ec0b04c61b2f80b3c1b23\nAuthor: Eric Wong \nDate: Sat Dec 15 19:08:22 2007 -0800\n\n git-svn: avoid leaving leftover committer/author info in rebase\n \n We set the 6 environment variables for controlling\n committer/author email/name/time for every commit.\n \n We do this in the parent process to be passed to\n git-commit-tree, because open3() doesn't afford us the control\n of doing it only in the child process. This means we leave them\n hanging around in the main process until the next revision comes\n around and all 6 environment variables are overwridden again.\n \n Unfortunately, for the last commit, leaving them hanging around\n means the git-rebase invocation will pick it up, rewriting the\n rebased commit with incorrect author information. This should fix\n it.\n \n Signed-off-by: Eric Wong \n---\n\n-- \nEric Wong\n"},{"id":"63734","messageId":"7vfxxzvxcu.fsf@gitster.siamese.dyndns.org","threadId":"11345","inReplyTo":"20071219084853.GB17787@muzzle","subject":"Re: git-svn fix for broken symlinks","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2007-12-19T09:37:37Z","receivedAt":"2007-12-19T09:37:37Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thanks. I was about to ask you about some patches I saw on the list\nfrom you.\n\nPulled; will push out tomorrow.\n"}]}