From: Mircea Bardac <dev@mircea.bardac.net>
To: git@vger.kernel.org
Cc: Fredrik Skolmli <fredrik@frsk.net>
Subject: Re: git svn --add-author-from implies --use-log-author
Date: Fri, 20 Jun 2008 14:31:32 +0100 [thread overview]
Message-ID: <485BB134.9080203@mircea.bardac.net> (raw)
In-Reply-To: <20080620113147.GC27940@frsk.net>
Hi,
Fredrik Skolmli wrote:
> when running git svn dcommit --add-author-from on a git-svn repository,
> --use-log-author is not implied by itself. This causes the rewritten history
> to not show the author in the way most users would expect it to be, "Name
> <email@domain.tld>", but instead "<user@uuid>".
>
> Instead of forcing people to write "svn dcommit --add-author-from
> --use-log-author", is it a bad move to imply the last argument, making the
> history look more user-friendly to start with?
The documentation mentions that these 2 parameters are for init and,
like other parameters, I had the feeling they will affect either the
"init" process or the entire existance of the repository. Now that I
look better, I see that there is no "config" note next to them.
From my point of view, they should be repository config options,
affecting all commits. I can't see a reason for having only sometimes
--add-author-from. If you want to commit information about the authors
in regards to a SVN repository, you'll always want that option to be on.
If you want to have authors properly identified when getting something
out of the SVN repository, you'll always want to have --use-log-author on.
I would also like some option to stop pushing a commit if it doesn't
have any From/Signed-off-by line in it.
Mircea
P.S. I am very curious why using From and not Signed-off-by. I had the
feeling that Signed-off-by is being used to also mark the path of a
patch. Also, is there a centralized list of all the ways a patch can be
"marked"
--
http://mircea.bardac.net
next prev parent reply other threads:[~2008-06-20 13:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-20 11:31 git svn --add-author-from implies --use-log-author Fredrik Skolmli
2008-06-20 13:31 ` Mircea Bardac [this message]
2008-06-23 20:12 ` Avery Pennarun
2008-06-23 21:22 ` Mircea Bardac
2008-06-23 21:31 ` Avery Pennarun
2008-06-24 7:37 ` Fredrik Skolmli
2008-06-24 16:02 ` [PATCH] git-svn docs: mention svn.addAuthorFrom and svn.useLogAuthor config options Avery Pennarun
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=485BB134.9080203@mircea.bardac.net \
--to=dev@mircea.bardac.net \
--cc=fredrik@frsk.net \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).