[PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[carlos.bilbao]

From: Carlos Bilbao <carlos.bilbao.osdev@gmail.com>

Update .mailmap to reflect my new (and final) primary email address,
carlos.bilbao@kernel.org. This ensures consistent attribution in Git
history. Also update my contact information in file
Documentation/translations/sp_SP/index.rst to help contributors reach out
for Spanish translations.

Signed-off-by: Carlos Bilbao <carlos.bilbao@kernel.org>
---
 .mailmap                                   | 3 +++
 Documentation/translations/sp_SP/index.rst | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/.mailmap b/.mailmap
index 5ff0e5d681e7..83837191affb 100644
--- a/.mailmap
+++ b/.mailmap
@@ -140,6 +140,9 @@ Cai Huoqing <cai.huoqing@linux.dev> <caihuoqing@baidu.com>
 Can Guo <quic_cang@quicinc.com> <cang@codeaurora.org>
 Carl Huang <quic_cjhuang@quicinc.com> <cjhuang@codeaurora.org>
 Carlos Bilbao <carlos.bilbao.osdev@gmail.com> <carlos.bilbao@amd.com>
+Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao.osdev@gmail.com>
+Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao@amd.com>
+Carlos Bilbao <carlos.bilbao@kernel.org> <bilbao@vt.edu>
 Changbin Du <changbin.du@intel.com> <changbin.du@gmail.com>
 Changbin Du <changbin.du@intel.com> <changbin.du@intel.com>
 Chao Yu <chao@kernel.org> <chao2.yu@samsung.com>
diff --git a/Documentation/translations/sp_SP/index.rst b/Documentation/translations/sp_SP/index.rst
index aae7018b0d1a..2b50283e1608 100644
--- a/Documentation/translations/sp_SP/index.rst
+++ b/Documentation/translations/sp_SP/index.rst
@@ -7,7 +7,7 @@ Traducción al español
 
 	\kerneldocCJKoff
 
-:maintainer: Carlos Bilbao <carlos.bilbao.osdev@gmail.com>
+:maintainer: Carlos Bilbao <carlos.bilbao@kernel.org>
 
 .. _sp_disclaimer:
 
-- 
2.43.5

Re: [PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[Jonathan Corbet]

carlos.bilbao@kernel.org writes:

> From: Carlos Bilbao <carlos.bilbao.osdev@gmail.com>
>
> Update .mailmap to reflect my new (and final) primary email address,
> carlos.bilbao@kernel.org. This ensures consistent attribution in Git
> history. Also update my contact information in file
> Documentation/translations/sp_SP/index.rst to help contributors reach out
> for Spanish translations.
>
> Signed-off-by: Carlos Bilbao <carlos.bilbao@kernel.org>
> ---
>  .mailmap                                   | 3 +++
>  Documentation/translations/sp_SP/index.rst | 2 +-
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/.mailmap b/.mailmap
> index 5ff0e5d681e7..83837191affb 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -140,6 +140,9 @@ Cai Huoqing <cai.huoqing@linux.dev> <caihuoqing@baidu.com>
>  Can Guo <quic_cang@quicinc.com> <cang@codeaurora.org>
>  Carl Huang <quic_cjhuang@quicinc.com> <cjhuang@codeaurora.org>
>  Carlos Bilbao <carlos.bilbao.osdev@gmail.com> <carlos.bilbao@amd.com>

Is there a reason you didn't just delete the above line?

> +Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao.osdev@gmail.com>
> +Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao@amd.com>
> +Carlos Bilbao <carlos.bilbao@kernel.org> <bilbao@vt.edu>
>  Changbin Du <changbin.du@intel.com> <changbin.du@gmail.com>

Thanks,

jon

Re: [PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[Carlos Bilbao]

Hello,

On 1/13/25 11:35, Jonathan Corbet wrote:
> carlos.bilbao@kernel.org writes:
>
>> From: Carlos Bilbao <carlos.bilbao.osdev@gmail.com>
>>
>> Update .mailmap to reflect my new (and final) primary email address,
>> carlos.bilbao@kernel.org. This ensures consistent attribution in Git
>> history. Also update my contact information in file
>> Documentation/translations/sp_SP/index.rst to help contributors reach out
>> for Spanish translations.
>>
>> Signed-off-by: Carlos Bilbao <carlos.bilbao@kernel.org>
>> ---
>>  .mailmap                                   | 3 +++
>>  Documentation/translations/sp_SP/index.rst | 2 +-
>>  2 files changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/.mailmap b/.mailmap
>> index 5ff0e5d681e7..83837191affb 100644
>> --- a/.mailmap
>> +++ b/.mailmap
>> @@ -140,6 +140,9 @@ Cai Huoqing <cai.huoqing@linux.dev> <caihuoqing@baidu.com>
>>  Can Guo <quic_cang@quicinc.com> <cang@codeaurora.org>
>>  Carl Huang <quic_cjhuang@quicinc.com> <cjhuang@codeaurora.org>
>>  Carlos Bilbao <carlos.bilbao.osdev@gmail.com> <carlos.bilbao@amd.com>
> Is there a reason you didn't just delete the above line?


My understanding was that .mailmap should keep older mappings for proper
record keeping, but it may have had the same effect, yes.


>
>> +Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao.osdev@gmail.com>
>> +Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao@amd.com>
>> +Carlos Bilbao <carlos.bilbao@kernel.org> <bilbao@vt.edu>
>>  Changbin Du <changbin.du@intel.com> <changbin.du@gmail.com>
> Thanks,
>
> jon


Thanks, Carlos

Re: [PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[Jonathan Corbet]

Carlos Bilbao <carlos.bilbao.osdev@gmail.com> writes:

> Hello,
>
> On 1/13/25 11:35, Jonathan Corbet wrote:
>> carlos.bilbao@kernel.org writes:
>>
>>> From: Carlos Bilbao <carlos.bilbao.osdev@gmail.com>
>>>
>>> Update .mailmap to reflect my new (and final) primary email address,
>>> carlos.bilbao@kernel.org. This ensures consistent attribution in Git
>>> history. Also update my contact information in file
>>> Documentation/translations/sp_SP/index.rst to help contributors reach out
>>> for Spanish translations.
>>>
>>> Signed-off-by: Carlos Bilbao <carlos.bilbao@kernel.org>
>>> ---
>>>  .mailmap                                   | 3 +++
>>>  Documentation/translations/sp_SP/index.rst | 2 +-
>>>  2 files changed, 4 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/.mailmap b/.mailmap
>>> index 5ff0e5d681e7..83837191affb 100644
>>> --- a/.mailmap
>>> +++ b/.mailmap
>>> @@ -140,6 +140,9 @@ Cai Huoqing <cai.huoqing@linux.dev> <caihuoqing@baidu.com>
>>>  Can Guo <quic_cang@quicinc.com> <cang@codeaurora.org>
>>>  Carl Huang <quic_cjhuang@quicinc.com> <cjhuang@codeaurora.org>
>>>  Carlos Bilbao <carlos.bilbao.osdev@gmail.com> <carlos.bilbao@amd.com>
>> Is there a reason you didn't just delete the above line?
>
>
> My understanding was that .mailmap should keep older mappings for proper
> record keeping, but it may have had the same effect, yes.

I'm confused, which records are we keeping here?  It doesn't seem that
mapping the same old address to multiple new ones will lead to anything
good...?

jon

Re: [PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[Carlos Bilbao]

On 1/19/25 12:03, Jonathan Corbet wrote:

> Carlos Bilbao <carlos.bilbao.osdev@gmail.com> writes:
>
>> Hello,
>>
>> On 1/13/25 11:35, Jonathan Corbet wrote:
>>> carlos.bilbao@kernel.org writes:
>>>
>>>> From: Carlos Bilbao <carlos.bilbao.osdev@gmail.com>
>>>>
>>>> Update .mailmap to reflect my new (and final) primary email address,
>>>> carlos.bilbao@kernel.org. This ensures consistent attribution in Git
>>>> history. Also update my contact information in file
>>>> Documentation/translations/sp_SP/index.rst to help contributors reach out
>>>> for Spanish translations.
>>>>
>>>> Signed-off-by: Carlos Bilbao <carlos.bilbao@kernel.org>
>>>> ---
>>>>  .mailmap                                   | 3 +++
>>>>  Documentation/translations/sp_SP/index.rst | 2 +-
>>>>  2 files changed, 4 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/.mailmap b/.mailmap
>>>> index 5ff0e5d681e7..83837191affb 100644
>>>> --- a/.mailmap
>>>> +++ b/.mailmap
>>>> @@ -140,6 +140,9 @@ Cai Huoqing <cai.huoqing@linux.dev> <caihuoqing@baidu.com>
>>>>  Can Guo <quic_cang@quicinc.com> <cang@codeaurora.org>
>>>>  Carl Huang <quic_cjhuang@quicinc.com> <cjhuang@codeaurora.org>
>>>>  Carlos Bilbao <carlos.bilbao.osdev@gmail.com> <carlos.bilbao@amd.com>
>>> Is there a reason you didn't just delete the above line?
>>
>> My understanding was that .mailmap should keep older mappings for proper
>> record keeping, but it may have had the same effect, yes.
> I'm confused, which records are we keeping here?  It doesn't seem that
> mapping the same old address to multiple new ones will lead to anything
> good...?


By records I meant mappings of older to newer emails, which in my head
is .mailmap purpose with version control. Do you think some of the
mappings as I've added them are redundant and/or problematic?


>
> jon


Thanks, Carlos

Re: [PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[Jonathan Corbet]

Carlos Bilbao <carlos.bilbao.osdev@gmail.com> writes:

> By records I meant mappings of older to newer emails, which in my head
> is .mailmap purpose with version control. Do you think some of the
> mappings as I've added them are redundant and/or problematic?

My point is that this line, which is in .mailmap now (and which your
patch preserves):

> Carlos Bilbao <carlos.bilbao.osdev@gmail.com> <carlos.bilbao@amd.com>

...says to map the @amd.com address to the gmail address.  But this new
line:

> Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao@amd.com>

says to map @amd.com to @kernel.org instead.  At best, this is
nonsensical; at worst, you may not get the full remapping of @amd.com
that you seem to want.  So ... why leave that line in place?

Thanks,

jon

Re: [PATCH] mailmap, docs: Update email to carlos.bilbao@kernel.org

[Carlos Bilbao]

Hello,

On 1/20/25 13:54, Jonathan Corbet wrote:
> Carlos Bilbao <carlos.bilbao.osdev@gmail.com> writes:
>
>> By records I meant mappings of older to newer emails, which in my head
>> is .mailmap purpose with version control. Do you think some of the
>> mappings as I've added them are redundant and/or problematic?
> My point is that this line, which is in .mailmap now (and which your
> patch preserves):
>
>> Carlos Bilbao <carlos.bilbao.osdev@gmail.com> <carlos.bilbao@amd.com>
> ...says to map the @amd.com address to the gmail address.  But this new
> line:
>
>> Carlos Bilbao <carlos.bilbao@kernel.org> <carlos.bilbao@amd.com>
> says to map @amd.com to @kernel.org instead.  At best, this is
> nonsensical; at worst, you may not get the full remapping of @amd.com
> that you seem to want.  So ... why leave that line in place?


That makes sense, thanks forclarifying. Sending out a patch correcting this
and updating myemail in MAINTAINERS now. Enjoy the weekend!
>
> Thanks,
>
> jon

Regards,
Carlos