From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: William Hubbs <williamh@gentoo.org>
Cc: git@vger.kernel.org, chutzpah@gentoo.org
Subject: Re: [PATCH v2 1/2] config: allow giving separate author and committer idents
Date: Fri, 25 Jan 2019 23:58:10 +0100 [thread overview]
Message-ID: <877ees4a65.fsf@evledraar.gmail.com> (raw)
In-Reply-To: <20190125215955.30032-2-williamh@gentoo.org>
On Fri, Jan 25 2019, William Hubbs wrote:
> The author.email, author.name, committer.email and committer.name
> settings are analogous to the GIT_AUTHOR_* and GIT_COMMITTER_*
> environment variables, but for the git config system. This allows them
> to be set separately for each repository.
>
> Git supports setting different authorship and committer
> information with environment variables. However, environment variables
> are set in the shell, so if different authorship and committer
> information is needed for different repositories an external tool is
> required.
>
> This adds support to git config for author.email, author.name,
> committer.email and committer.name settings so this information
> can be set per repository.
>
> Also, it generalizes the fmt_ident function so it can handle author vs
> committer identification.
I have not tested this in any detail...
> diff --git a/Documentation/config/user.txt b/Documentation/config/user.txt
> index b5b2ba1199..18e1ec3c1b 100644
> --- a/Documentation/config/user.txt
> +++ b/Documentation/config/user.txt
> @@ -1,12 +1,39 @@
> +author.email::
> + The email address used for the author of newly
> + created commits. Defaults to the value of the
> + `GIT_AUTHOR_EMAIL` environment variable, or if
> + the environment variable is not set, the `user.email`
> + configuration variable.
> +
> +author.name::
> + The full name used for the author of newly created commits.
> + Defaults to the value of the `GIT_AUTHOR_NAME` environment variable, or
> + if the environment variable is not set,
> + the `user.email` configuration variable.
> +
> +committer.email::
> + The email address used for the committer of newly created commits.
> + Defaults to the value of the `GIT_COMMITTER_EMAIL` environment
> + variable, or if the environment variable is not set, the `user.email`
> + configuration variable.
> +
> +committer.name::
> + The full name used for the committer of newly created commits.
> + Defaults to the value of the `GIT_COMMITTER_NAME` environment
> + variable, or if the environment variable is not set, the `user.name`
> + configuration variable.
> +
> user.email::
> Your email address to be recorded in any newly created commits.
> Can be overridden by the `GIT_AUTHOR_EMAIL`, `GIT_COMMITTER_EMAIL`, and
> - `EMAIL` environment variables. See linkgit:git-commit-tree[1].
> + `EMAIL` environment variables or the `author.email` or
> + `committer.email` settings discussed above. See linkgit:git-commit-tree[1].
>
> user.name::
> Your full name to be recorded in any newly created commits.
> Can be overridden by the `GIT_AUTHOR_NAME` and `GIT_COMMITTER_NAME`
> - environment variables. See linkgit:git-commit-tree[1].
> + environment variables or the `author.name` or `committer.name`
> + settings discussed above. See linkgit:git-commit-tree[1].
Looks correct, although I wonder if we're at the point where it would be
better to present this info as a table.
> diff --git a/builtin/am.c b/builtin/am.c
> index 95370313b6..53fdd22c45 100644
> --- a/builtin/am.c
> +++ b/builtin/am.c
> @@ -1594,7 +1594,7 @@ static void do_commit(const struct am_state *state)
> }
>
> author = fmt_ident(state->author_name, state->author_email,
> - state->ignore_date ? NULL : state->author_date,
> + WANT_AUTHOR_IDENT, state->ignore_date ? NULL : state->author_date,
This & a few other things in this series take the code beyond 79
characters.
> IDENT_STRICT);
>
> if (state->committer_date_is_author_date)
> diff --git a/builtin/commit.c b/builtin/commit.c
> index 004b816635..f96b90daeb 100644
> --- a/builtin/commit.c
> +++ b/builtin/commit.c
> @@ -607,7 +607,8 @@ static void determine_author_info(struct strbuf *author_ident)
> set_ident_var(&date, strbuf_detach(&date_buf, NULL));
> }
>
> - strbuf_addstr(author_ident, fmt_ident(name, email, date, IDENT_STRICT));
> + strbuf_addstr(author_ident, fmt_ident(name, email, WANT_AUTHOR_IDENT, date,
> + IDENT_STRICT));
> assert_split_ident(&author, author_ident);
> export_one("GIT_AUTHOR_NAME", author.name_begin, author.name_end, 0);
> export_one("GIT_AUTHOR_EMAIL", author.mail_begin, author.mail_end, 0);
> diff --git a/cache.h b/cache.h
> index 49713cc5a5..f13c6e244f 100644
> --- a/cache.h
> +++ b/cache.h
> @@ -1479,10 +1479,19 @@ int date_overflows(timestamp_t date);
> #define IDENT_STRICT 1
> #define IDENT_NO_DATE 2
> #define IDENT_NO_NAME 4
> +
> +enum want_ident {
> + WANT_BLANK_IDENT,
> + WANT_AUTHOR_IDENT,
> + WANT_COMMITTER_IDENT,
> +};
> +
> extern const char *git_author_info(int);
> extern const char *git_committer_info(int);
> -extern const char *fmt_ident(const char *name, const char *email, const char *date_str, int);
> -extern const char *fmt_name(const char *name, const char *email);
> +extern const char *fmt_ident(const char *name, const char *email,
> + enum want_ident whose_ident,
> + const char *date_str, int);
> +extern const char *fmt_name(enum want_ident);
> extern const char *ident_default_name(void);
> extern const char *ident_default_email(void);
> extern const char *git_editor(void);
> diff --git a/config.c b/config.c
> index ff521eb27a..4bd5920dea 100644
> --- a/config.c
> +++ b/config.c
> @@ -1484,6 +1484,12 @@ int git_default_config(const char *var, const char *value, void *cb)
> return 0;
> }
>
> + if (starts_with(var, "author."))
> + return git_ident_config(var, value, cb);
> +
> + if (starts_with(var, "committer."))
> + return git_ident_config(var, value, cb);
> +
> /* Add other config variables here and to Documentation/config.txt. */
> return 0;
> }
> diff --git a/ident.c b/ident.c
> index 33bcf40644..a1e774c3e5 100644
> --- a/ident.c
> +++ b/ident.c
> @@ -11,6 +11,10 @@
> static struct strbuf git_default_name = STRBUF_INIT;
> static struct strbuf git_default_email = STRBUF_INIT;
> static struct strbuf git_default_date = STRBUF_INIT;
> +static struct strbuf git_author_name = STRBUF_INIT;
> +static struct strbuf git_author_email = STRBUF_INIT;
> +static struct strbuf git_committer_name = STRBUF_INIT;
> +static struct strbuf git_committer_email = STRBUF_INIT;
> static int default_email_is_bogus;
> static int default_name_is_bogus;
>
> @@ -355,13 +359,19 @@ N_("\n"
> "\n");
>
> const char *fmt_ident(const char *name, const char *email,
> - const char *date_str, int flag)
> + enum want_ident whose_ident, const char *date_str, int flag)
> {
> static struct strbuf ident = STRBUF_INIT;
> int strict = (flag & IDENT_STRICT);
> int want_date = !(flag & IDENT_NO_DATE);
> int want_name = !(flag & IDENT_NO_NAME);
>
> + if (!email) {
> + if (whose_ident == WANT_AUTHOR_IDENT && git_author_email.len)
> + email = git_author_email.buf;
> + else if (whose_ident == WANT_COMMITTER_IDENT && git_committer_email.len)
> + email = git_committer_email.buf;
> + }
> if (!email) {
> if (strict && ident_use_config_only
> && !(ident_config_given & IDENT_MAIL_GIVEN)) {
> @@ -377,6 +387,13 @@ const char *fmt_ident(const char *name, const char *email,
>
> if (want_name) {
> int using_default = 0;
> + if (!name) {
> + if (whose_ident == WANT_AUTHOR_IDENT && git_author_name.len)
> + name = git_author_name.buf;
> + else if (whose_ident == WANT_COMMITTER_IDENT &&
> + git_committer_name.len)
> + name = git_committer_name.buf;
> + }
> if (!name) {
> if (strict && ident_use_config_only
> && !(ident_config_given & IDENT_NAME_GIVEN)) {
> @@ -425,9 +442,25 @@ const char *fmt_ident(const char *name, const char *email,
> return ident.buf;
> }
>
> -const char *fmt_name(const char *name, const char *email)
> +const char *fmt_name(enum want_ident whose_ident)
> {
> - return fmt_ident(name, email, NULL, IDENT_STRICT | IDENT_NO_DATE);
> + char *name = NULL;
> + char *email = NULL;
> +
> + switch (whose_ident) {
> + case WANT_BLANK_IDENT:
> + break;
> + case WANT_AUTHOR_IDENT:
> + name = getenv("GIT_AUTHOR_NAME");
> + email = getenv("GIT_AUTHOR_EMAIL");
> + break;
> + case WANT_COMMITTER_IDENT:
> + name = getenv("GIT_COMMITTER_NAME");
> + email = getenv("GIT_COMMITTER_EMAIL");
> + break;
> + }
> + return fmt_ident(name, email, whose_ident, NULL,
> + IDENT_STRICT | IDENT_NO_DATE);
> }
>
> const char *git_author_info(int flag)
> @@ -438,6 +471,7 @@ const char *git_author_info(int flag)
> author_ident_explicitly_given |= IDENT_MAIL_GIVEN;
> return fmt_ident(getenv("GIT_AUTHOR_NAME"),
> getenv("GIT_AUTHOR_EMAIL"),
> + WANT_AUTHOR_IDENT,
> getenv("GIT_AUTHOR_DATE"),
> flag);
> }
> @@ -450,6 +484,7 @@ const char *git_committer_info(int flag)
> committer_ident_explicitly_given |= IDENT_MAIL_GIVEN;
> return fmt_ident(getenv("GIT_COMMITTER_NAME"),
> getenv("GIT_COMMITTER_EMAIL"),
> + WANT_COMMITTER_IDENT,
> getenv("GIT_COMMITTER_DATE"),
> flag);
> }
> @@ -480,6 +515,46 @@ int git_ident_config(const char *var, const char *value, void *data)
> return 0;
> }
>
> + if (!strcmp(var, "author.name")) {
> + if (!value)
> + return config_error_nonbool(var);
> + strbuf_reset(&git_author_name);
> + strbuf_addstr(&git_author_name, value);
> + author_ident_explicitly_given |= IDENT_NAME_GIVEN;
> + ident_config_given |= IDENT_NAME_GIVEN;
> + return 0;
> + }
> +
> + if (!strcmp(var, "author.email")) {
> + if (!value)
> + return config_error_nonbool(var);
> + strbuf_reset(&git_author_email);
> + strbuf_addstr(&git_author_email, value);
> + author_ident_explicitly_given |= IDENT_MAIL_GIVEN;
> + ident_config_given |= IDENT_MAIL_GIVEN;
> + return 0;
> + }
> +
> + if (!strcmp(var, "committer.name")) {
> + if (!value)
> + return config_error_nonbool(var);
> + strbuf_reset(&git_committer_name);
> + strbuf_addstr(&git_committer_name, value);
> + committer_ident_explicitly_given |= IDENT_NAME_GIVEN;
> + ident_config_given |= IDENT_NAME_GIVEN;
> + return 0;
> + }
> +
> + if (!strcmp(var, "committer.email")) {
> + if (!value)
> + return config_error_nonbool(var);
> + strbuf_reset(&git_committer_email);
> + strbuf_addstr(&git_committer_email, value);
> + committer_ident_explicitly_given |= IDENT_MAIL_GIVEN;
> + ident_config_given |= IDENT_MAIL_GIVEN;
> + return 0;
> + }
> +
This whole thing should be split into a static function. It's the same
code copy/pasted 4x times just with a differnet value for "var", the
strbuf variable & IDENT_*_GIVEN.
next prev parent reply other threads:[~2019-01-25 22:58 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-25 21:59 Add author and committer configuration settings William Hubbs
2019-01-25 21:59 ` [PATCH v2 1/2] config: allow giving separate author and committer idents William Hubbs
2019-01-25 22:58 ` Ævar Arnfjörð Bjarmason [this message]
2019-01-28 18:58 ` William Hubbs
2019-01-28 19:00 ` Junio C Hamano
2019-01-28 20:04 ` Ævar Arnfjörð Bjarmason
2019-01-28 21:33 ` Junio C Hamano
2019-01-28 23:30 ` William Hubbs
2019-01-29 22:42 ` William Hubbs
2019-01-25 21:59 ` [PATCH v2 2/2] tests: add test for " William Hubbs
2019-01-25 23:05 ` Ævar Arnfjörð Bjarmason
2019-01-26 1:06 ` William Hubbs
2019-01-26 8:53 ` Ævar Arnfjörð Bjarmason
2019-01-27 4:48 ` Eric Sunshine
2019-01-28 19:05 ` Ævar Arnfjörð Bjarmason
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
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=877ees4a65.fsf@evledraar.gmail.com \
--to=avarab@gmail.com \
--cc=chutzpah@gentoo.org \
--cc=git@vger.kernel.org \
--cc=williamh@gentoo.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.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
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 read-only IMAP folder(s) and NNTP newsgroup(s).