{"thread":{"id":"40184","subject":"git clone svn: authors from authors file are ignored, authors-prog works, but crashes on branch points","startedAt":"2015-08-26T19:57:10Z","lastAt":"2015-09-11T08:36:55Z","messageCount":7,"participants":["Till Schäfer","Michael J Gruber","Eric Wong","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"268754","messageId":"5613050.3arVUQYvEz@granit","threadId":"40184","inReplyTo":null,"subject":"git clone svn: authors from authors file are ignored, authors-prog works, but crashes on branch points","fromName":"Till Schäfer","fromEmail":"till2.schaefer@tu-dortmund.de","sentAt":"2015-08-26T19:57:10Z","receivedAt":"2015-08-26T19:57:10Z","isPatch":false,"sender":{"key":"till2.schaefer@tu-dortmund.de","avatar":null},"body":"Hi,\ni am observing some weired \"git svn clone\" behavior during my try to migrate the Scaffold Hunter [1] SVN repository [2] to Git: \n\nif i just use the command \n\n$ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --no-metadata -s code\n\neverything went smoothly. Now i wanted to replace the authors SVN-logins by useful names and created a mapping file with the following content: \n\nanjenson = Andrew Zhilka <>\nbernhard.dick = Bernhard Dick <>\ndominic.sacre = Dominic Sacré <>\ndoxmoxbox = doxmoxbox <>\nfalkn = Falk Nette <>\nhenning.garus = Henning Garus <>\nkakl = Karsten Klein <>\nklein = Karsten Klein <>\nlappie00 = Jeroen Lappenschaar <>\nmichael.hesse = Michael Hesse <>\nnlskrg = Nils Kriege \nphilipp.kopp = Philipp Kopp <>\nphilipp.lewe = Philipp Lewe <>\nschrins = Sven Schrinner <>\nshamshadnpti = Shamshad Alam <>\nsrenner7 = Steffen Renner <>\nsturm89 = Werner Sturm <>\nthomas.schmitz = Thomas Schmitz <>\nthorsten.fluegel = Thorsten Flügel <>\ntill.schaefer = Till Schäfer \ntillschaefer = Till Schäfer \n\n\nI executed:\n\n$ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --authors-file=/home/till/temp/code-authors-transform.txt --no-metadata -s code\n\nand got the error message \"Author: klein not defined in /home/till/temp/code-authors-transform.txt file\". \nI tried a workaround by using a python script to return the entries i have already defined in the authors file: \n\n$ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --authors-file=/home/till/temp/code-authors-transform.txt --authors-prog=/home/till/temp/authors.py --no-metadata -s code\n\nThe import process now went over the \"klein\" commit. The weired thing is, that a few SVN usernames seem to be recognized in the authors file and a few are passed to my script (I logged the output). However, at the first revision, where a tag was added in the SVN repo the cloning process crashed with the error message: \n\nFound possible branch point: svn://svn.code.sf.net/p/scaffoldhunter/code/trunk => svn://svn.code.sf.net/p/scaffoldhunter/code/branches/subsearch, 17\nUse of uninitialized value $u in substitution (s///) at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\nUse of uninitialized value $u in concatenation (.) or string at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\nrefs/remotes/origin/trunk: 'svn://svn.code.sf.net/p/scaffoldhunter/code' not found in ''\n\nOne problem I am seeing in the SVN repo is, that between the revisions 97 and 102 the trunk folder was absent. the old one was moved to some subfolder and a new one was created. Therefore, I started the cloning beginning with rev 102 using the command line option \"-r102:HEAD\". However, the same error occurred for some later tag (the first tag after rev 102). \n\nFound possible branch point: svn://svn.code.sf.net/p/scaffoldhunter/code/trunk => svn://svn.code.sf.net/p/scaffoldhunter/code/tags/release-2.0, 1565\nUse of uninitialized value $u in substitution (s///) at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\nUse of uninitialized value $u in concatenation (.) or string at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\nrefs/remotes/origin/trunk: 'svn://svn.code.sf.net/p/scaffoldhunter/code' not found in ''\n\nI would be very glad if someone has a hint about what is going wrong here. Is this a bug in git or is something wrong with the SVN repo?\n\n\nused software versions: \n- git 2.5.0 \n- subversion 1.8.14\n\n\nRegards,\nTill\n\n[1] http://scaffoldhunter.sourceforge.net/\n[2] https://svn.code.sf.net/p/scaffoldhunter/code/trunk\n\n-- \nDipl.-Inf. Till Schäfer\nTU Dortmund University\nChair 11 - Algorithm Engineering\nOtto-Hahn-Str. 14 / Room 237\n44227 Dortmund, Germany\n\ne-mail: till.schaefer@cs.tu-dortmund.de\nphone: +49(231)755-7706\nfax: +49(231)755-7740\nweb: http://ls11-www.cs.uni-dortmund.de/staff/schaefer\npgp: https://keyserver2.pgp.com/vkd/SubmitSearch.event?&&SearchCriteria=0xD84DED79\n"},{"id":"269683","messageId":"55F17847.9020401@drmicha.warpmail.net","threadId":"40184","inReplyTo":"5613050.3arVUQYvEz@granit","subject":"Re: git clone svn: authors from authors file are ignored, authors-prog works, but crashes on branch points","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2015-09-10T12:32:07Z","receivedAt":"2015-09-10T12:32:07Z","isPatch":false,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Till Schäfer venit, vidit, dixit 26.08.2015 21:57:\n> Hi,\n> i am observing some weired \"git svn clone\" behavior during my try to migrate the Scaffold Hunter [1] SVN repository [2] to Git: \n> \n> if i just use the command \n> \n> $ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --no-metadata -s code\n\nIs there a specific reason for \"--no-metadata\"? The man page advises\nagainst it. Basically, it keeps you from converting the history in\nchunks. (You can always filter-branch everything after a complete, \"one\nshot\" conversion which may be done in stages.)\n\n> everything went smoothly. Now i wanted to replace the authors SVN-logins by useful names and created a mapping file with the following content: \n> \n> anjenson = Andrew Zhilka <>\n> bernhard.dick = Bernhard Dick <>\n> dominic.sacre = Dominic Sacré <>\n> doxmoxbox = doxmoxbox <>\n> falkn = Falk Nette <>\n> henning.garus = Henning Garus <>\n> kakl = Karsten Klein <>\n> klein = Karsten Klein <>\n> lappie00 = Jeroen Lappenschaar <>\n> michael.hesse = Michael Hesse <>\n> nlskrg = Nils Kriege \n> philipp.kopp = Philipp Kopp <>\n> philipp.lewe = Philipp Lewe <>\n> schrins = Sven Schrinner <>\n> shamshadnpti = Shamshad Alam <>\n> srenner7 = Steffen Renner <>\n> sturm89 = Werner Sturm <>\n> thomas.schmitz = Thomas Schmitz <>\n> thorsten.fluegel = Thorsten Flügel <>\n> till.schaefer = Till Schäfer \n> tillschaefer = Till Schäfer \n> \n> \n> I executed:\n> \n> $ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --authors-file=/home/till/temp/code-authors-transform.txt --no-metadata -s code\n> \n> and got the error message \"Author: klein not defined in /home/till/temp/code-authors-transform.txt file\". \n> I tried a workaround by using a python script to return the entries i have already defined in the authors file: \n> \n> $ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --authors-file=/home/till/temp/code-authors-transform.txt --authors-prog=/home/till/temp/authors.py --no-metadata -s code\n> \n> The import process now went over the \"klein\" commit. The weired thing is, that a few SVN usernames seem to be recognized in the authors file and a few are passed to my script (I logged the output). However, at the first revision, where a tag was added in the SVN repo the cloning process crashed with the error message: \n\nThat first part is weird already. The reason is that author-file\nprocessing and author-prog output processing use different perl regexps.\nI will follow up with a patch.\n\n> Found possible branch point: svn://svn.code.sf.net/p/scaffoldhunter/code/trunk => svn://svn.code.sf.net/p/scaffoldhunter/code/branches/subsearch, 17\n> Use of uninitialized value $u in substitution (s///) at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> Use of uninitialized value $u in concatenation (.) or string at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> refs/remotes/origin/trunk: 'svn://svn.code.sf.net/p/scaffoldhunter/code' not found in ''\n> \n> One problem I am seeing in the SVN repo is, that between the revisions 97 and 102 the trunk folder was absent. the old one was moved to some subfolder and a new one was created. Therefore, I started the cloning beginning with rev 102 using the command line option \"-r102:HEAD\". However, the same error occurred for some later tag (the first tag after rev 102). \n> \n> Found possible branch point: svn://svn.code.sf.net/p/scaffoldhunter/code/trunk => svn://svn.code.sf.net/p/scaffoldhunter/code/tags/release-2.0, 1565\n> Use of uninitialized value $u in substitution (s///) at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> Use of uninitialized value $u in concatenation (.) or string at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> refs/remotes/origin/trunk: 'svn://svn.code.sf.net/p/scaffoldhunter/code' not found in ''\n> \n> I would be very glad if someone has a hint about what is going wrong here. Is this a bug in git or is something wrong with the SVN repo?\n\nThe layout change (no root) certainly doesn't help. We shouldn't crash,\nthough.\n\n> used software versions: \n> - git 2.5.0 \n> - subversion 1.8.14\n\nThe conversion goes through for me with (your other options and) your\nauthors file and specifying a stupid authors prog:\n\n--->8---\n#!/bin/sh\ngrep \"$1\" /tmp/t/authors | head -n 1 | cut -d'=' -f2 | cut -c'2-'\n--->8---\n\nand:\n\ngit version 2.6.0.rc0.178.g4177fa5\nsvn, Version 1.8.13 (r1667537)\n\nwith our without --authors-file...\n\n(It went also through with a patched git-svn and just the authors file.)\n\nI don't think git-svn has changed since git 2.5.0. Maybe you can\ndowngrade svn for the one-shot conversion? I don't see anything\nsuspicious in their changelog but don't remember it as being overly\ncomplete.\n\ngit svn served us well in making itself mostly obsolete. The downside is\nthat not many are using it any more. Keeping up with subversion changes\n(to keep git svn working) has become more and more difficult.\n\nMichael\n"},{"id":"269682","messageId":"03f07c11135aef9e04a26e5b1018d726ba2fab5d.1441887195.git.git@drmicha.warpmail.net","threadId":"40184","inReplyTo":"5613050.3arVUQYvEz@granit","subject":"[PATCH] git-svn: parse authors file more leniently","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2015-09-10T12:32:13Z","receivedAt":"2015-09-10T12:32:13Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Currently, git-svn parses an authors file using the perl regex\n\n/^(.+?|\\(no author\\))\\s*=\\s*(.+?)\\s*<(.+)>\\s*$/\n\nin order to extract svn user name, real name and e-mail.\nThis does not match an empty e-mail field like \"<>\". On the other hand,\nthe output of an authors-prog is parsed with the perl regex\n\n/^\\s*(.+?)\\s*<(.*)>\\s*$/\n\nin order to extract real name and e-mail.\n\nSo, specifying a trivial file grep such as\n\ngrep \"$1\" /tmp/authors | head -n 1 | cut -d'=' -f2 | cut -c'2-'\n\nas the authors prog gives different results compared to specifying\n/tmp/authors as the authors file directly.\n\nInstead, make git svn uses the perl regex\n\n/^(.+?|\\(no author\\))\\s*=\\s*(.+?)\\s*<(.*)>\\s*$/\n\nfor parsing the authors file so that the same (slightly more lenient)\nregex is used in both cases.\n\nReported-by: Till Schäfer \nSigned-off-by: Michael J Gruber \n---\n git-svn.perl | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/git-svn.perl b/git-svn.perl\nindex 36f7240..fa5f253 100755\n--- a/git-svn.perl\n+++ b/git-svn.perl\n@@ -1924,7 +1924,7 @@ sub load_authors {\n \tmy $log = $cmd eq 'log';\n \twhile (<$authors>) {\n \t\tchomp;\n-\t\tnext unless /^(.+?|\\(no author\\))\\s*=\\s*(.+?)\\s*<(.+)>\\s*$/;\n+\t\tnext unless /^(.+?|\\(no author\\))\\s*=\\s*(.+?)\\s*<(.*)>\\s*$/;\n \t\tmy ($user, $name, $email) = ($1, $2, $3);\n \t\tif ($log) {\n \t\t\t$Git::SVN::Log::rusers{\"$name <$email>\"} = $user;\n-- \n2.6.0.rc0.178.g4177fa5\n"},{"id":"269713","messageId":"5258505.eFJUESI73n@granit","threadId":"40184","inReplyTo":"55F17847.9020401@drmicha.warpmail.net","subject":"Re: git clone svn: authors from authors file are ignored, authors-prog works, but crashes on branch points","fromName":"Till Schäfer","fromEmail":"till2.schaefer@tu-dortmund.de","sentAt":"2015-09-10T17:29:37Z","receivedAt":"2015-09-10T17:29:37Z","isPatch":false,"sender":{"key":"till2.schaefer@tu-dortmund.de","avatar":null},"body":"Hi, \nI can confirm, that the patch (see mail: \"[PATCH] git-svn: parse authors file more leniently\") fixes the \"Author: **** not defined in **** file\" problem in my case. \n\nDowngrading to subversion 1.8.13 (from 1.8.14) also fixes the branching crash in revision 17. With subversion 1.8.14 i still get: \n\nFound possible branch point: svn://svn.code.sf.net/p/scaffoldhunter/code/trunk => svn://svn.code.sf.net/p/scaffoldhunter/code/branches/subsearch, 17\nUse of uninitialized value $u in substitution (s///) at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\nUse of uninitialized value $u in concatenation (.) or string at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\nrefs/remotes/origin/trunk: 'svn://svn.code.sf.net/p/scaffoldhunter/code' not found in ''\n\n\nAm Donnerstag, 10. September 2015, 14:32:07 schrieb Michael J Gruber:\n> Till Schäfer venit, vidit, dixit 26.08.2015 21:57:\n> > Hi,\n> > i am observing some weired \"git svn clone\" behavior during my try to migrate the Scaffold Hunter [1] SVN repository [2] to Git: \n> > \n> > if i just use the command \n> > \n> > $ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --no-metadata -s code\n> \n> Is there a specific reason for \"--no-metadata\"? The man page advises\n> against it. Basically, it keeps you from converting the history in\n> chunks. (You can always filter-branch everything after a complete, \"one\n> shot\" conversion which may be done in stages.)\n\ni was basically following the tutorial at git-scm [1] (which is also somewhat outdated in the later svn branch conversion). Anyway, i did not read the manual :-/ (man-page)\n\n\nRegards, \nTill\n\n[1] https://git-scm.com/book/en/v2/Git-and-Other-Systems-Migrating-to-Git\n\n\n\n\n\n\n\n\n\n\n> \n> > everything went smoothly. Now i wanted to replace the authors SVN-logins by useful names and created a mapping file with the following content: \n> > \n> > anjenson = Andrew Zhilka <>\n> > bernhard.dick = Bernhard Dick <>\n> > dominic.sacre = Dominic Sacré <>\n> > doxmoxbox = doxmoxbox <>\n> > falkn = Falk Nette <>\n> > henning.garus = Henning Garus <>\n> > kakl = Karsten Klein <>\n> > klein = Karsten Klein <>\n> > lappie00 = Jeroen Lappenschaar <>\n> > michael.hesse = Michael Hesse <>\n> > nlskrg = Nils Kriege \n> > philipp.kopp = Philipp Kopp <>\n> > philipp.lewe = Philipp Lewe <>\n> > schrins = Sven Schrinner <>\n> > shamshadnpti = Shamshad Alam <>\n> > srenner7 = Steffen Renner <>\n> > sturm89 = Werner Sturm <>\n> > thomas.schmitz = Thomas Schmitz <>\n> > thorsten.fluegel = Thorsten Flügel <>\n> > till.schaefer = Till Schäfer \n> > tillschaefer = Till Schäfer \n> > \n> > \n> > I executed:\n> > \n> > $ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --authors-file=/home/till/temp/code-authors-transform.txt --no-metadata -s code\n> > \n> > and got the error message \"Author: klein not defined in /home/till/temp/code-authors-transform.txt file\". \n> > I tried a workaround by using a python script to return the entries i have already defined in the authors file: \n> > \n> > $ git svn clone svn://svn.code.sf.net/p/scaffoldhunter/code --authors-file=/home/till/temp/code-authors-transform.txt --authors-prog=/home/till/temp/authors.py --no-metadata -s code\n> > \n> > The import process now went over the \"klein\" commit. The weired thing is, that a few SVN usernames seem to be recognized in the authors file and a few are passed to my script (I logged the output). However, at the first revision, where a tag was added in the SVN repo the cloning process crashed with the error message: \n> \n> That first part is weird already. The reason is that author-file\n> processing and author-prog output processing use different perl regexps.\n> I will follow up with a patch.\n> \n> > Found possible branch point: svn://svn.code.sf.net/p/scaffoldhunter/code/trunk => svn://svn.code.sf.net/p/scaffoldhunter/code/branches/subsearch, 17\n> > Use of uninitialized value $u in substitution (s///) at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> > Use of uninitialized value $u in concatenation (.) or string at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> > refs/remotes/origin/trunk: 'svn://svn.code.sf.net/p/scaffoldhunter/code' not found in ''\n> > \n> > One problem I am seeing in the SVN repo is, that between the revisions 97 and 102 the trunk folder was absent. the old one was moved to some subfolder and a new one was created. Therefore, I started the cloning beginning with rev 102 using the command line option \"-r102:HEAD\". However, the same error occurred for some later tag (the first tag after rev 102). \n> > \n> > Found possible branch point: svn://svn.code.sf.net/p/scaffoldhunter/code/trunk => svn://svn.code.sf.net/p/scaffoldhunter/code/tags/release-2.0, 1565\n> > Use of uninitialized value $u in substitution (s///) at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> > Use of uninitialized value $u in concatenation (.) or string at /usr/lib64/perl5/vendor_perl/5.20.2/Git/SVN.pm line 101.\n> > refs/remotes/origin/trunk: 'svn://svn.code.sf.net/p/scaffoldhunter/code' not found in ''\n> > \n> > I would be very glad if someone has a hint about what is going wrong here. Is this a bug in git or is something wrong with the SVN repo?\n> \n> The layout change (no root) certainly doesn't help. We shouldn't crash,\n> though.\n> \n> > used software versions: \n> > - git 2.5.0 \n> > - subversion 1.8.14\n> \n> The conversion goes through for me with (your other options and) your\n> authors file and specifying a stupid authors prog:\n> \n> --->8---\n> #!/bin/sh\n> grep \"$1\" /tmp/t/authors | head -n 1 | cut -d'=' -f2 | cut -c'2-'\n> --->8---\n> \n> and:\n> \n> git version 2.6.0.rc0.178.g4177fa5\n> svn, Version 1.8.13 (r1667537)\n> \n> with our without --authors-file...\n> \n> (It went also through with a patched git-svn and just the authors file.)\n> \n> I don't think git-svn has changed since git 2.5.0. Maybe you can\n> downgrade svn for the one-shot conversion? I don't see anything\n> suspicious in their changelog but don't remember it as being overly\n> complete.\n> \n> git svn served us well in making itself mostly obsolete. The downside is\n> that not many are using it any more. Keeping up with subversion changes\n> (to keep git svn working) has become more and more difficult.\n> \n> Michael\n\n\n\n\n\n-- \nDipl.-Inf. Till Schäfer\nTU Dortmund University\nChair 11 - Algorithm Engineering\nOtto-Hahn-Str. 14 / Room 237\n44227 Dortmund, Germany\n\ne-mail: till.schaefer@cs.tu-dortmund.de\nphone: +49(231)755-7706\nfax: +49(231)755-7740\nweb: http://ls11-www.cs.uni-dortmund.de/staff/schaefer\npgp: https://keyserver2.pgp.com/vkd/SubmitSearch.event?&&SearchCriteria=0xD84DED79\n"},{"id":"269722","messageId":"20150910180810.GA22112@dcvr.yhbt.net","threadId":"40184","inReplyTo":"03f07c11135aef9e04a26e5b1018d726ba2fab5d.1441887195.git.git@drmicha.warpmail.net","subject":"Re: [PATCH] git-svn: parse authors file more leniently","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2015-09-10T18:08:10Z","receivedAt":"2015-09-10T18:08:10Z","isPatch":true,"sender":{"key":"e@80x24.org","avatar":null},"body":"Michael J Gruber wrote:\n> Instead, make git svn uses the perl regex\n> \n> /^(.+?|\\(no author\\))\\s*=\\s*(.+?)\\s*<(.*)>\\s*$/\n> \n> for parsing the authors file so that the same (slightly more lenient)\n> regex is used in both cases.\n> \n> Reported-by: Till Schäfer \n> Signed-off-by: Michael J Gruber \n\nThanks.\nSigned-off-by: Eric Wong \n\nAnd pushed to master of git://bogomips.org/git-svn\n(commit f7c6de0ea1bd5722a1181c6279676c6831b38a34)\n\nBy the way, I also had some other patches sitting around for you.\nDid you ever have time to revisit them? (I haven't)\n\n t/lib-httpd: load mod_unixd\n t/lib-git-svn: check same httpd module dirs as lib-httpd\n"},{"id":"269741","messageId":"xmqqwpvy0yop.fsf@gitster.mtv.corp.google.com","threadId":"40184","inReplyTo":"20150910180810.GA22112@dcvr.yhbt.net","subject":"Re: [PATCH] git-svn: parse authors file more leniently","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2015-09-10T21:08:54Z","receivedAt":"2015-09-10T21:08:54Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Eric Wong writes:\n\n> Michael J Gruber wrote:\n>> Instead, make git svn uses the perl regex\n>> \n>> /^(.+?|\\(no author\\))\\s*=\\s*(.+?)\\s*<(.*)>\\s*$/\n>> \n>> for parsing the authors file so that the same (slightly more lenient)\n>> regex is used in both cases.\n>> \n>> Reported-by: Till Schäfer \n>> Signed-off-by: Michael J Gruber \n>\n> Thanks.\n> Signed-off-by: Eric Wong \n>\n> And pushed to master of git://bogomips.org/git-svn\n> (commit f7c6de0ea1bd5722a1181c6279676c6831b38a34)\n\nThanks, pulled.\n"},{"id":"269756","messageId":"55F292A7.1030809@drmicha.warpmail.net","threadId":"40184","inReplyTo":"20150910180810.GA22112@dcvr.yhbt.net","subject":"Re: [PATCH] git-svn: parse authors file more leniently","fromName":"Michael J Gruber","fromEmail":"git@drmicha.warpmail.net","sentAt":"2015-09-11T08:36:55Z","receivedAt":"2015-09-11T08:36:55Z","isPatch":true,"sender":{"key":"git@grubix.eu","avatar":"https://avatars.githubusercontent.com/u/233215?v=4"},"body":"Eric Wong venit, vidit, dixit 10.09.2015 20:08:\n> Michael J Gruber wrote:\n>> Instead, make git svn uses the perl regex\n>>\n>> /^(.+?|\\(no author\\))\\s*=\\s*(.+?)\\s*<(.*)>\\s*$/\n>>\n>> for parsing the authors file so that the same (slightly more lenient)\n>> regex is used in both cases.\n>>\n>> Reported-by: Till Schäfer \n>> Signed-off-by: Michael J Gruber \n> \n> Thanks.\n> Signed-off-by: Eric Wong \n> \n> And pushed to master of git://bogomips.org/git-svn\n> (commit f7c6de0ea1bd5722a1181c6279676c6831b38a34)\n> \n> By the way, I also had some other patches sitting around for you.\n> Did you ever have time to revisit them? (I haven't)\n> \n> t/lib-httpd: load mod_unixd\n> t/lib-git-svn: check same httpd module dirs as lib-httpd\n> \n\nAlso \"from me\".\n\nShort answer: No\n\nIf I remember correctly, they were correct bit not complete in the sense\nthat on a standard Fedora install (with newer apache), svn tests still\nwouldn't run over http. But I/we learned that those tests were simply\nrun over local file protocol instead when svn over http didn't work. On\na standard debian install (which apparantly has non-standard, thus\ndownwards compatible apache config) everything was fine with or without\nthose patches.\n\nI still plan to look at them when I find time. (I'll be retiring\nsometime between 20 and 30 years from now, so there's hope.)\n\nMichael\n"}]}