Sasha Khapyorsky <sashak@xxxxxxxxxxxx> writes: > Hi, > > This feature was useful with importing https://openib.org/svn/gen2 . > > Sasha > > This add '-S' option. When specified svn-import will try to parse > commit message for 'Signed-off-by: ...' line, and if found will use > the name and email address extracted at first occurrence as this commit > author name and author email address. Committer name and email are > extracted in usual way. > > Signed-off-by: Sasha Khapyorsky <sashak@xxxxxxxxxxxx> Thanks. I do not think the first signed-off-by is necessarily the author of the change, so we are risking miscrediting (or misblaming) a wrong person. Having said that, using the committer information has the same miscredit problem, so this change might be Ok, but I am not sure if it adds much improvement. I will wait for an ack/nack from somebody who use svnimport and know it well. BTW, I do not appreciate the first six lines of your message being _before_ the proposed commit log message. Please have it between "---\n" (that comes immediately after your own "Signed-off-by:") and the diffstat, like this: This add '-S' option. When specified svn-import will try to... ... in usual way. Signed-off-by: Sasha Khapyorsky <sashak@xxxxxxxxxxxx> --- Hi, This feature was useful with importing https://openib.org/svn/gen2 . Sasha git-svnimport.perl | 31 ++++++++++++++++++++----------- 1 files changed, 20 insertions(+), 11 deletions(-) diff --git a/... - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html