Map svn to git authors in Git mirrors

[Ryan J Ollos]

This is a request for a change to Trac's Git mirrors that are hosted on GitHub and BitBucket. It looks like we could use an ​authors files when cloning the SVN repository in order to get accurate svn to git username mapping. I didn't see this discussed in #9235. Has anyone tried this before?

git svn clone -A authors.txt http://svn.edgewall.org/repos/trac

For the mirrors on GitHub, this should result in the Git user ids being linked to our GitHub accounts, provided we have the email address registered with our GitHub account.

[Ryan J Ollos]

authors.txt​ has the authors that were exported from the SVN repository.

[Jun Omae]

I looked into name and email of the authors. See attachment:authors-v2.txt​.

[Christian Boos]

From ​authors files:

Not only does the Author field look a lot better, but the git-svn-id is no longer there, either.

I'm not sure we want this to happen in our case, as we use that line to easily figure out which Subversion revision this is about, and the fact that when it's not there we immediately know it's a Git-only changeset.

[Ryan J Ollos]

Replying to cboos:

I'm not sure we want this to happen in our case, as we use that line to easily figure out which Subversion revision this is about, and the fact that when it's not there we immediately know it's a Git-only changeset.

Yeah I can definitely see the value in keeping the git-svn-id lines. I had to read the section you referenced a few times, and then confirmed behavior by cloning. They've confusingly interleaved discussion of the --authors flag and --no-metdata flag. As long as we don't pass the --no-metadata flag, we'll still get the git-svn-id in the log message.

I didn't want to spend all day cloning the whole repository, so what I actually did was (building from ​gdiscussion:trac-dev:eN8rscI2BPE):

$ mkdir teo-clone
$ cd teo-clone
$ git svn init --prefix=mirror/ -s --rewrite-root="http://trac.edgewall.org/intertrac/log:" https://svn.edgewall.org/repos/trac
$ git svn fetch -A authors-v2.txt -r 13000:HEAD

$ git log -n 2
commit 0a18397444e51a532612811baf7d25454f743533
Author: Jun Omae <jun66j5@gmail.com>
Date:   Thu Aug 14 03:25:09 2014 +0000

    1.1.2dev: merged [13068] from 1.0-stable (fix for #11715)
    
    
    git-svn-id: http://trac.edgewall.org/intertrac/log:/trunk@13069 af82e41b-90c

commit aac7c286730f7cff1dcddfc18e6fbcc2a2044d5e
Author: Ryan Ollos <ryan.j.ollos@gmail.com>
Date:   Wed Aug 6 10:46:49 2014 +0000

    1.1.2dev: Fixed regression in [13055]. Refs #11697.
    
    Permission checks were being performed on a invalid realm.
    
    
    git-svn-id: http://trac.edgewall.org/intertrac/log:/trunk@13067 af82e41b-90c

From the current mirrors, the output is:

t$ git log -n 2
commit f77545ea5001deb54e02258c4e2e9c77c12126eb
Author: jomae <jomae@af82e41b-90c4-0310-8c96-b1721e28e2e2>
Date:   Thu Aug 14 03:25:09 2014 +0000

    1.1.2dev: merged [13068] from 1.0-stable (fix for #11715)
    
    
    git-svn-id: http://trac.edgewall.org/intertrac/log:/trunk@13069 af82e41b-90c

commit 99c9541e7fb0e399674a20cf9cc8e175281d502a
Author: rjollos <rjollos@af82e41b-90c4-0310-8c96-b1721e28e2e2>
Date:   Wed Aug 6 10:46:49 2014 +0000

    1.1.2dev: Fixed regression in [13055]. Refs #11697.
    
    Permission checks were being performed on a invalid realm.
    
    
    git-svn-id: http://trac.edgewall.org/intertrac/log:/trunk@13067 af82e41b-90c

[Ryan J Ollos]

This is linked from our internal tracker, in an issue that proposes to use SubGit for git-svn mirroring. This issue will likely be addressed if we ever decide to recreate the mirrors.