* Re: [PATCH v2] mailmap: update brian m. carlson's email address
2018-05-08 1:58 ` [PATCH v2] " brian m. carlson
@ 2018-05-08 3:21 ` Junio C Hamano
2018-05-08 5:04 ` Kaartic Sivaraam
2018-05-22 22:08 ` Jonathan Nieder
2 siblings, 0 replies; 18+ messages in thread
From: Junio C Hamano @ 2018-05-08 3:21 UTC (permalink / raw)
To: brian m. carlson; +Cc: git
"brian m. carlson" <sandals@crustytoothpaste.net> writes:
> An earlier change, cdb6b5ac (".mailmap: Combine more (name, email) to
> individual persons", 2013-08-12), noted that there were two name
> spellings and two email addresses and mapped the crustytoothpaste.net
> address to the crustytoothpaste.ath.cx address. The latter is an older,
> obsolete address, while the former is current, so switch the order of
> the addresses so that git log displays the correct address.
>
> Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
> ---
> I intentionally avoided the use of the first person here, because I
> wasn't sure what the preference of the list was on that. Hopefully it
> reads naturally and isn't awkward.
>
> If not, I can send a v3.
Nah, the updated text is perfect. Thanks, will replace.
>
> .mailmap | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/.mailmap b/.mailmap
> index 7c71e88ea5..df7cf6313c 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -25,8 +25,8 @@ Ben Walton <bdwalton@gmail.com> <bwalton@artsci.utoronto.ca>
> Benoit Sigoure <tsunanet@gmail.com> <tsuna@lrde.epita.fr>
> Bernt Hansen <bernt@norang.ca> <bernt@alumni.uwaterloo.ca>
> Brandon Casey <drafnel@gmail.com> <casey@nrlssc.navy.mil>
> -brian m. carlson <sandals@crustytoothpaste.ath.cx> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
> -brian m. carlson <sandals@crustytoothpaste.ath.cx> <sandals@crustytoothpaste.net>
> +brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
> +brian m. carlson <sandals@crustytoothpaste.net> <sandals@crustytoothpaste.ath.cx>
> Bryan Larsen <bryan@larsen.st> <bryan.larsen@gmail.com>
> Bryan Larsen <bryan@larsen.st> <bryanlarsen@yahoo.com>
> Cheng Renquan <crquan@gmail.com>
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: update brian m. carlson's email address
2018-05-08 1:58 ` [PATCH v2] " brian m. carlson
2018-05-08 3:21 ` Junio C Hamano
@ 2018-05-08 5:04 ` Kaartic Sivaraam
2018-05-09 0:19 ` brian m. carlson
2018-05-22 22:08 ` Jonathan Nieder
2 siblings, 1 reply; 18+ messages in thread
From: Kaartic Sivaraam @ 2018-05-08 5:04 UTC (permalink / raw)
To: brian m. carlson, Junio C Hamano; +Cc: git
[-- Attachment #1.1: Type: text/plain, Size: 1893 bytes --]
On Tuesday 08 May 2018 07:28 AM, brian m. carlson wrote:
> An earlier change, cdb6b5ac (".mailmap: Combine more (name, email) to
> individual persons", 2013-08-12), noted that there were two name
> spellings and two email addresses and mapped the crustytoothpaste.net
> address to the crustytoothpaste.ath.cx address. The latter is an older,
> obsolete address, while the former is current, so switch the order of
> the addresses so that git log displays the correct address.
>
Just to be sure, you're meaning the use of `git log --use-mailmap` when
you mean `git log` in the log message. Am I right? Or did you mean `git
shortlog` ?
I'm asking this because I think the `git log` output doesn't consider
the mailmap file by default.
--
Sivaraam
QUOTE:
“The most valuable person on any team is the person who makes everyone
else on the team more valuable, not the person who knows the most.”
- Joel Spolsky
Sivaraam?
You possibly might have noticed that my signature recently changed from
'Kaartic' to 'Sivaraam' both of which are parts of my name. I find the
new signature to be better for several reasons one of which is that the
former signature has a lot of ambiguities in the place I live as it is a
common name (NOTE: it's not a common spelling, just a common name). So,
I switched signatures before it's too late.
That said, I won't mind you calling me 'Kaartic' if you like it [of
course ;-)]. You can always call me using either of the names.
KIND NOTE TO THE NATIVE ENGLISH SPEAKER:
As I'm not a native English speaker myself, there might be mistaeks in
my usage of English. I apologise for any mistakes that I make.
It would be "helpful" if you take the time to point out the mistakes.
It would be "super helpful" if you could provide suggestions about how
to correct those mistakes.
Thanks in advance!
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: update brian m. carlson's email address
2018-05-08 5:04 ` Kaartic Sivaraam
@ 2018-05-09 0:19 ` brian m. carlson
2018-05-09 7:03 ` Kaartic Sivaraam
0 siblings, 1 reply; 18+ messages in thread
From: brian m. carlson @ 2018-05-09 0:19 UTC (permalink / raw)
To: Kaartic Sivaraam; +Cc: Junio C Hamano, git
[-- Attachment #1: Type: text/plain, Size: 538 bytes --]
On Tue, May 08, 2018 at 10:34:58AM +0530, Kaartic Sivaraam wrote:
> Just to be sure, you're meaning the use of `git log --use-mailmap` when
> you mean `git log` in the log message. Am I right? Or did you mean `git
> shortlog` ?
>
> I'm asking this because I think the `git log` output doesn't consider
> the mailmap file by default.
Correct, it doesn't. In my case, I was using --pretty='%aN <%aE>',
which is how I noticed it in the first place.
--
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 867 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: update brian m. carlson's email address
2018-05-09 0:19 ` brian m. carlson
@ 2018-05-09 7:03 ` Kaartic Sivaraam
2018-05-10 23:13 ` brian m. carlson
0 siblings, 1 reply; 18+ messages in thread
From: Kaartic Sivaraam @ 2018-05-09 7:03 UTC (permalink / raw)
To: brian m. carlson, Junio C Hamano, git
[-- Attachment #1.1: Type: text/plain, Size: 334 bytes --]
On Wednesday 09 May 2018 05:49 AM, brian m. carlson wrote:
> Correct, it doesn't. In my case, I was using --pretty='%aN <%aE>',
> which is how I noticed it in the first place.
So, how about updating the commit message to avoid confusions to the
incidental future reader? (Or is it just not worth it? ;-])
--
Sivaraam
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: update brian m. carlson's email address
2018-05-09 7:03 ` Kaartic Sivaraam
@ 2018-05-10 23:13 ` brian m. carlson
0 siblings, 0 replies; 18+ messages in thread
From: brian m. carlson @ 2018-05-10 23:13 UTC (permalink / raw)
To: Kaartic Sivaraam; +Cc: Junio C Hamano, git
[-- Attachment #1: Type: text/plain, Size: 687 bytes --]
On Wed, May 09, 2018 at 12:33:59PM +0530, Kaartic Sivaraam wrote:
> On Wednesday 09 May 2018 05:49 AM, brian m. carlson wrote:
> > Correct, it doesn't. In my case, I was using --pretty='%aN <%aE>',
> > which is how I noticed it in the first place.
>
> So, how about updating the commit message to avoid confusions to the
> incidental future reader? (Or is it just not worth it? ;-])
I don't think it's worth it. It does apply to git log, just not unless
you enable the mailmap, which I supposed was implicit. I can do a v3 if
someone feels strongly, but otherwise I think it's fine as it is.
--
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 867 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: update brian m. carlson's email address
2018-05-08 1:58 ` [PATCH v2] " brian m. carlson
2018-05-08 3:21 ` Junio C Hamano
2018-05-08 5:04 ` Kaartic Sivaraam
@ 2018-05-22 22:08 ` Jonathan Nieder
2018-05-22 22:42 ` brian m. carlson
2 siblings, 1 reply; 18+ messages in thread
From: Jonathan Nieder @ 2018-05-22 22:08 UTC (permalink / raw)
To: brian m. carlson; +Cc: Junio C Hamano, git
Hi,
brian m. carlson wrote:
> An earlier change, cdb6b5ac (".mailmap: Combine more (name, email) to
> individual persons", 2013-08-12), noted that there were two name
> spellings and two email addresses and mapped the crustytoothpaste.net
> address to the crustytoothpaste.ath.cx address. The latter is an older,
> obsolete address, while the former is current, so switch the order of
> the addresses so that git log displays the correct address.
>
> Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
> ---
> I intentionally avoided the use of the first person here, because I
> wasn't sure what the preference of the list was on that. Hopefully it
> reads naturally and isn't awkward.
I have no preference. First or third person would work equally well
for me for this kind of thing.
[...]
> --- a/.mailmap
> +++ b/.mailmap
> @@ -25,8 +25,8 @@ Ben Walton <bdwalton@gmail.com> <bwalton@artsci.utoronto.ca>
> Benoit Sigoure <tsunanet@gmail.com> <tsuna@lrde.epita.fr>
> Bernt Hansen <bernt@norang.ca> <bernt@alumni.uwaterloo.ca>
> Brandon Casey <drafnel@gmail.com> <casey@nrlssc.navy.mil>
> -brian m. carlson <sandals@crustytoothpaste.ath.cx> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
> -brian m. carlson <sandals@crustytoothpaste.ath.cx> <sandals@crustytoothpaste.net>
> +brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
> +brian m. carlson <sandals@crustytoothpaste.net> <sandals@crustytoothpaste.ath.cx>
Today I tried a "git shortlog" run and was surprised to find
Brian M. Carlson (6):
Add missing test file for UTF-16.
submodule: fix confusing variable name
submodule: don't print status output with ignore=all
send-email: don't call methods on undefined values
http-backend: provide Allow header for 405
remote-curl: fix large pushes with GSSAPI
These commits use author Brian M. Carlson <sandals@crustytoothpaste.net>.
Previously they matched
brian m. carlson <sandals@crustytoothpaste.ath.cx> <sandals@crustytoothpaste.net>
but now they don't match any line in the .mailmap file.
Should we add a line
brian m. carlson <sandals@crustytoothpaste.net>
to handle these commits?
Thanks,
Jonathan
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: update brian m. carlson's email address
2018-05-22 22:08 ` Jonathan Nieder
@ 2018-05-22 22:42 ` brian m. carlson
2018-09-17 18:18 ` [PATCH] mailmap: consistently normalize brian m. carlson's name Jonathan Nieder
0 siblings, 1 reply; 18+ messages in thread
From: brian m. carlson @ 2018-05-22 22:42 UTC (permalink / raw)
To: Jonathan Nieder; +Cc: Junio C Hamano, git
[-- Attachment #1: Type: text/plain, Size: 551 bytes --]
On Tue, May 22, 2018 at 03:08:26PM -0700, Jonathan Nieder wrote:
> These commits use author Brian M. Carlson <sandals@crustytoothpaste.net>.
> Previously they matched
>
> brian m. carlson <sandals@crustytoothpaste.ath.cx> <sandals@crustytoothpaste.net>
>
> but now they don't match any line in the .mailmap file.
>
> Should we add a line
>
> brian m. carlson <sandals@crustytoothpaste.net>
>
> to handle these commits?
Ah, good point. Probably so.
--
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 867 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] mailmap: consistently normalize brian m. carlson's name
2018-05-22 22:42 ` brian m. carlson
@ 2018-09-17 18:18 ` Jonathan Nieder
2018-09-17 22:21 ` brian m. carlson
0 siblings, 1 reply; 18+ messages in thread
From: Jonathan Nieder @ 2018-09-17 18:18 UTC (permalink / raw)
To: brian m. carlson; +Cc: Junio C Hamano, git
v2.18.0-rc0~70^2 (mailmap: update brian m. carlson's email address,
2018-05-08) changed the mailmap to map
sandals@crustytoothpaste.ath.cx
-> brian m. carlson <sandals@crustytoothpaste.net>
instead of
sandals@crustytoothpaste.net
-> brian m. carlson <sandals@crustytoothpaste.ath.cx>
That means the mapping
Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
-> brian m. carlson <sandals@crustytoothpaste.net>
is redundant, so we can remove it. More importantly, it means that
the identity "Brian M. Carlson <sandals@crustytoothpaste.net>" used in
some commits is not normalized any more. Add a mapping for it.
Noticed while updating Debian's Git packaging, which uses "git
shortlog --no-merges" to produce a list of changes in each version,
grouped by author's (normalized) name.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
brian m. carlson wrote:
> On Tue, May 22, 2018 at 03:08:26PM -0700, Jonathan Nieder wrote:
>> These commits use author Brian M. Carlson <sandals@crustytoothpaste.net>.
>> Previously they matched
>>
>> brian m. carlson <sandals@crustytoothpaste.ath.cx> <sandals@crustytoothpaste.net>
>>
>> but now they don't match any line in the .mailmap file.
>>
>> Should we add a line
>>
>> brian m. carlson <sandals@crustytoothpaste.net>
>>
>> to handle these commits?
>
> Ah, good point. Probably so.
Sorry to leave this hanging for so long. How about this patch?
Thanks,
Jonathan
.mailmap | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.mailmap b/.mailmap
index f165222a78..29047a110a 100644
--- a/.mailmap
+++ b/.mailmap
@@ -25,7 +25,7 @@ Ben Walton <bdwalton@gmail.com> <bwalton@artsci.utoronto.ca>
Benoit Sigoure <tsunanet@gmail.com> <tsuna@lrde.epita.fr>
Bernt Hansen <bernt@norang.ca> <bernt@alumni.uwaterloo.ca>
Brandon Casey <drafnel@gmail.com> <casey@nrlssc.navy.mil>
-brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
+brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.net>
brian m. carlson <sandals@crustytoothpaste.net> <sandals@crustytoothpaste.ath.cx>
Bryan Larsen <bryan@larsen.st> <bryan.larsen@gmail.com>
Bryan Larsen <bryan@larsen.st> <bryanlarsen@yahoo.com>
--
2.19.0.397.gdd90340f6a
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH] mailmap: consistently normalize brian m. carlson's name
2018-09-17 18:18 ` [PATCH] mailmap: consistently normalize brian m. carlson's name Jonathan Nieder
@ 2018-09-17 22:21 ` brian m. carlson
2018-09-24 17:39 ` [PATCH v2] " Jonathan Nieder
0 siblings, 1 reply; 18+ messages in thread
From: brian m. carlson @ 2018-09-17 22:21 UTC (permalink / raw)
To: Jonathan Nieder; +Cc: Junio C Hamano, git
[-- Attachment #1: Type: text/plain, Size: 1181 bytes --]
On Mon, Sep 17, 2018 at 11:18:00AM -0700, Jonathan Nieder wrote:
> v2.18.0-rc0~70^2 (mailmap: update brian m. carlson's email address,
> 2018-05-08) changed the mailmap to map
>
> sandals@crustytoothpaste.ath.cx
> -> brian m. carlson <sandals@crustytoothpaste.net>
>
> instead of
>
> sandals@crustytoothpaste.net
> -> brian m. carlson <sandals@crustytoothpaste.ath.cx>
>
> That means the mapping
>
> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
> -> brian m. carlson <sandals@crustytoothpaste.net>
>
> is redundant, so we can remove it. More importantly, it means that
> the identity "Brian M. Carlson <sandals@crustytoothpaste.net>" used in
> some commits is not normalized any more. Add a mapping for it.
>
> Noticed while updating Debian's Git packaging, which uses "git
> shortlog --no-merges" to produce a list of changes in each version,
> grouped by author's (normalized) name.
I think this commit message makes sense. I apparently still fail to
understand how the .mailmap format works, so I can't tell you if the
patch is correct.
--
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 868 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH v2] mailmap: consistently normalize brian m. carlson's name
2018-09-17 22:21 ` brian m. carlson
@ 2018-09-24 17:39 ` Jonathan Nieder
2018-09-25 1:15 ` brian m. carlson
0 siblings, 1 reply; 18+ messages in thread
From: Jonathan Nieder @ 2018-09-24 17:39 UTC (permalink / raw)
To: brian m. carlson; +Cc: Junio C Hamano, git
v2.18.0-rc0~70^2 (mailmap: update brian m. carlson's email address,
2018-05-08) changed the mailmap to map
sandals@crustytoothpaste.ath.cx
-> brian m. carlson <sandals@crustytoothpaste.net>
instead of
sandals@crustytoothpaste.net
-> brian m. carlson <sandals@crustytoothpaste.ath.cx>
That means the mapping
Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
-> brian m. carlson <sandals@crustytoothpaste.net>
is redundant, so we can remove it. More importantly, it means that
the identity "Brian M. Carlson <sandals@crustytoothpaste.net>" used in
some commits is not normalized any more. Add a mapping for it.
Noticed while updating Debian's Git packaging, which uses "git
shortlog --no-merges" to produce a list of changes in each version,
grouped by author's (normalized) name.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
Hi,
brian m. carlson wrote:
> I think this commit message makes sense. I apparently still fail to
> understand how the .mailmap format works, so I can't tell you if the
> patch is correct.
Thanks for looking it over. What would it take to make the patch make
sense, too? ;-)
Most mailmap entries are of the form
Some Name <someemail@example.com>
which means "Wherever you see the email address someemail@example.com,
canonicalize the author's name to Some Name". We can use that:
brian m. carlson <sandals@crustytoothpaste.net>
When we see sandals@crustytoothpaste.ath.cx, we also want to
canonicalize the email address. For that, we can do
brian m. carlson <sandals@crustytoothpaste.net> <sandals@crustytoothpaste.ath.cx>
There's only one person who has used these email addresses, so we
don't have to do matching by name. If we wanted to tighten the name
normalization to match by name, I think we'd do something like
brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson
but I can't get that to seem to have any effect when I test with the
"git check-mailmap" command --- for example, "git check-mailmap 'Dana
How <random.email@example.com>'" does not map and "git check-mailmap
'Random Name <danahow@gmail.com>'" maps to 'Dana L. How
<danahow@gmail.com>'.
The even tighter matching used in v1
brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.net>
does work, but it's unnecessary complexity. We don't need it.
How about this?
Changes since v1:
- loosened the matching to only look at email and ignore name
- no other changes
.mailmap | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.mailmap b/.mailmap
index f165222a78..bef3352b0d 100644
--- a/.mailmap
+++ b/.mailmap
@@ -25,7 +25,7 @@ Ben Walton <bdwalton@gmail.com> <bwalton@artsci.utoronto.ca>
Benoit Sigoure <tsunanet@gmail.com> <tsuna@lrde.epita.fr>
Bernt Hansen <bernt@norang.ca> <bernt@alumni.uwaterloo.ca>
Brandon Casey <drafnel@gmail.com> <casey@nrlssc.navy.mil>
-brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
+brian m. carlson <sandals@crustytoothpaste.net>
brian m. carlson <sandals@crustytoothpaste.net> <sandals@crustytoothpaste.ath.cx>
Bryan Larsen <bryan@larsen.st> <bryan.larsen@gmail.com>
Bryan Larsen <bryan@larsen.st> <bryanlarsen@yahoo.com>
--
2.19.0.444.g18242da7ef
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: consistently normalize brian m. carlson's name
2018-09-24 17:39 ` [PATCH v2] " Jonathan Nieder
@ 2018-09-25 1:15 ` brian m. carlson
2018-09-25 2:13 ` Jonathan Nieder
0 siblings, 1 reply; 18+ messages in thread
From: brian m. carlson @ 2018-09-25 1:15 UTC (permalink / raw)
To: Jonathan Nieder; +Cc: Junio C Hamano, git
[-- Attachment #1: Type: text/plain, Size: 3149 bytes --]
On Mon, Sep 24, 2018 at 10:39:02AM -0700, Jonathan Nieder wrote:
> Hi,
>
> brian m. carlson wrote:
>
> > I think this commit message makes sense. I apparently still fail to
> > understand how the .mailmap format works, so I can't tell you if the
> > patch is correct.
>
> Thanks for looking it over. What would it take to make the patch make
> sense, too? ;-)
I certainly didn't mean to imply a failing on your part for explaining
the change adequately. I've just always found the format confusing and
I know others do, too.
> Most mailmap entries are of the form
>
> Some Name <someemail@example.com>
>
> which means "Wherever you see the email address someemail@example.com,
> canonicalize the author's name to Some Name". We can use that:
>
> brian m. carlson <sandals@crustytoothpaste.net>
>
> When we see sandals@crustytoothpaste.ath.cx, we also want to
> canonicalize the email address. For that, we can do
>
> brian m. carlson <sandals@crustytoothpaste.net> <sandals@crustytoothpaste.ath.cx>
>
> There's only one person who has used these email addresses, so we
> don't have to do matching by name. If we wanted to tighten the name
> normalization to match by name, I think we'd do something like
>
> brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson
>
> but I can't get that to seem to have any effect when I test with the
> "git check-mailmap" command --- for example, "git check-mailmap 'Dana
> How <random.email@example.com>'" does not map and "git check-mailmap
> 'Random Name <danahow@gmail.com>'" maps to 'Dana L. How
> <danahow@gmail.com>'.
>
> The even tighter matching used in v1
>
> brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.net>
>
> does work, but it's unnecessary complexity. We don't need it.
This has been a really helpful explanation. Thanks.
Maybe I'll have some time over the next week or so to send a patch to
the documentation to make it more understandable to past me.
> How about this?
>
> Changes since v1:
> - loosened the matching to only look at email and ignore name
> - no other changes
>
> .mailmap | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.mailmap b/.mailmap
> index f165222a78..bef3352b0d 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -25,7 +25,7 @@ Ben Walton <bdwalton@gmail.com> <bwalton@artsci.utoronto.ca>
> Benoit Sigoure <tsunanet@gmail.com> <tsuna@lrde.epita.fr>
> Bernt Hansen <bernt@norang.ca> <bernt@alumni.uwaterloo.ca>
> Brandon Casey <drafnel@gmail.com> <casey@nrlssc.navy.mil>
> -brian m. carlson <sandals@crustytoothpaste.net> Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
> +brian m. carlson <sandals@crustytoothpaste.net>
> brian m. carlson <sandals@crustytoothpaste.net> <sandals@crustytoothpaste.ath.cx>
> Bryan Larsen <bryan@larsen.st> <bryan.larsen@gmail.com>
> Bryan Larsen <bryan@larsen.st> <bryanlarsen@yahoo.com>
Having read your explanation, this looks good. Thanks for fixing this.
--
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 868 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] mailmap: consistently normalize brian m. carlson's name
2018-09-25 1:15 ` brian m. carlson
@ 2018-09-25 2:13 ` Jonathan Nieder
0 siblings, 0 replies; 18+ messages in thread
From: Jonathan Nieder @ 2018-09-25 2:13 UTC (permalink / raw)
To: brian m. carlson; +Cc: Junio C Hamano, git
brian m. carlson wrote:
> On Mon, Sep 24, 2018 at 10:39:02AM -0700, Jonathan Nieder wrote:
>> brian m. carlson wrote:
>>> I think this commit message makes sense.
[...]
>> What would it take to make the patch make
>> sense, too? ;-)
>
> I certainly didn't mean to imply a failing on your part for explaining
> the change adequately. I've just always found the format confusing and
> I know others do, too.
No worries. I took the opportunity because the patch isn't in "next"
yet so I was looking for a way to nudge it forward. I think v2 is
simpler than v1. Thanks for your help.
[...]
> This has been a really helpful explanation. Thanks.
>
> Maybe I'll have some time over the next week or so to send a patch to
> the documentation to make it more understandable to past me.
Sounds good. I think the (non-)handling of the 'name <email> name'
case is likely to be a bug.
>> How about this?
[...]
> Having read your explanation, this looks good. Thanks for fixing this.
\o/ Thanks for looking it over.
Sincerely,
Jonathan
^ permalink raw reply [flat|nested] 18+ messages in thread