Message-ID: <bb3fe216-9fe3-cc64-decf-290df128e370@amd.com>
Date:   Tue, 7 Mar 2023 07:41:51 -0600
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.1
Subject: Re: [PATCH] docs/sp_SP: Add process deprecated translation
Content-Language: en-US
To:     Akira Yokosawa <akiyks@gmail.com>
Cc:     corbet@lwn.net, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, sergio.collado@gmail.com
References: <20230306134420.30210-1-carlos.bilbao@amd.com>
 <38cb9f23-a56a-f420-5942-0bfeb620306e@gmail.com>
 <875b2564-589b-c381-cbf0-f30470d4a5a8@amd.com>
 <6f1d8954-586b-efea-526b-ef461b5624df@gmail.com>
 <2b4f177b-3b8a-fef6-e7a5-692db347679e@amd.com>
 <dbd8ece3-fb92-9ca3-917b-4951af7e1593@gmail.com>
From:   Carlos Bilbao <carlos.bilbao@amd.com>
In-Reply-To: <dbd8ece3-fb92-9ca3-917b-4951af7e1593@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QWdodlo4Wk5keXhKTU8rZndsSUl5R0FMMDRwMm9XZGlZcjYxc0hrS3Nsd2VH?=
 =?utf-8?B?VDVLaTAxRW5hR1ZDVkpKNGtZM2FBWS9yUWJ0aTZnQ0JlaXFuSmdDSzB3VVY4?=
 =?utf-8?B?UGg0WDZ6MERNQmJOS0puUyttbjhScTJyVTYvYkNleE8xY0E0Y25mRTQ4T001?=
 =?utf-8?B?MGRoVWJhS0FNWlV0a2xVRWlZcFU4QUpRMzk0aWdPRzBaSDliSVJNVFo4ZGgy?=
 =?utf-8?B?Uy9jZlJoUm1YanZHbzdEbHNicDZ5TWVKVmFtZ1ZEM3ZEMmxUcEFNT2NKSUpn?=
 =?utf-8?B?SSs1OGVFUDAwT0U5d2M2Y2UyL3RxZGZIMXdYWUFuNS9kMmI5Z3JIdHplUVUw?=
 =?utf-8?B?WnhVVkVsclRpVTZSRnhvVDBrTmZkZDdjRzZhNFYyamlrZU1UVGJ6SVB3MmRI?=
 =?utf-8?B?dFhmVTUxYmpWV3BNQkcycWVYcW85Y3l4bEpET2l5K043aWxVdXAwdml3UnRv?=
 =?utf-8?B?cTBsNHhBczVIOUJaYURRY3luY3psVFNxQ3QxeFplMnBuR1kyK0pwUEcrek5p?=
 =?utf-8?B?QXB4NWtFQkt4c1gyY1pKM29iNFprZDdZTUUyRXllMGpHVXhkajRPaThQZHFk?=
 =?utf-8?B?MTBWVDFyVVJQUzVoTTFDZElrMDdLZ2lzRGhVNlk1eXdpNjJtUnVuUG1rd1RO?=
 =?utf-8?B?SGxuOFVuUk8rTlB6S0ZXVks0ZTdxSGJPNVhsMCtxWmdXK3IwMW4wa0QxbWgz?=
 =?utf-8?B?K25xbEZWTXQwUHh2endMa3NNb084L29xYy8vTGMxVmMvNTZrNHo3U1FIRng5?=
 =?utf-8?B?Slk3UVJVY0RkVTBiRmRqdXZnQkx6bVQ3bjlIelB2V2Q4UGZENlY1amIvMU9m?=
 =?utf-8?B?RGdxckFQTWxHNzZZYWY0aGpuVVIyR2VxNWdKbFhvbVJpUFA1Y1lDRzl4Yi94?=
 =?utf-8?B?VVQ5azdyaisvQ28zU09rN1doMzR2ZUoyRzRrR1Z2UHhkWFZ3MDlKbG5KQjl0?=
 =?utf-8?B?ZU1iQkI2c3FUbnd1bG1wcFdLM0NXcENHVDFUZjlOaEU1NkcrK3ZCWFJuUzlT?=
 =?utf-8?B?OW9HWmtHTWpzbUNsbWhXbTJCY1g4b3BKTkZWaGlkdjlVY2hJZXFXbWFoME90?=
 =?utf-8?B?QzJSSjk3Mm9nc0NMYXJFblp1QWw1amlUQXR4NXBmZVVxalVDN09QVk5wQlpU?=
 =?utf-8?B?MHY5T3dscGdGWGJEa2RiRXgrZllSWGdhTGZ4NmxUYTd1c2Z5TGlKZDFtMlRu?=
 =?utf-8?B?WUFYRDgrT3lKUU1nY2hCMC9jd0ZqSGZ4Sjk3VlF2Vzl5dnBkSGI5cS9HOWdm?=
 =?utf-8?B?TmlxQUlBRmdQTDBUU0phUVI2T2VnNnFjV210ZTFDYmtkbFFabXlkeXo2Mi9I?=
 =?utf-8?B?YVR2SEsrR0QxVlRwTzk0WEZBcjBrRW9mOFFzRFRMV1Rxd1JkNE45Q2pkaENm?=
 =?utf-8?B?QXdOeExYSjUzVng0L29DN0QzYXcxamZzVzR5b2JzeHc3QkJPMElhVTFMcFZG?=
 =?utf-8?B?a3FOZmdJaGVlaUFvemNSM2ZTVkVCd3Y2OG5YT01TYVJsdHFYelpUM3Bha0NQ?=
 =?utf-8?B?YXRoODI5SHpneHZFeExZRllGTjVtM2xaYTVxTVBvWkRQMlVTS0RHZEE0ZStw?=
 =?utf-8?B?Wnp1VkYvYloySXBRL1VJV0tZcUpzcWZVOWRXRW9URktFQTVRZ2Irc2xJaWcv?=
 =?utf-8?B?VlpXNDdIUG9HcUU5VGllcmtmbm5pTktXQmFZM3dFZTlDWWRDTUQxZ1BNSkRG?=
 =?utf-8?B?YmZSZkJrOVR6cFQvL0RnNVlES0tZejgzS0JoNXNtT255WWgzQ3ppRnp1ek8y?=
 =?utf-8?B?WmxrbTA0ZUZSRmhpaVhCZHdwakorSFVsN3NnZWZmNDFGK1B4Z2V3WTNHbzNy?=
 =?utf-8?B?SjVQY3ljUFd0SXdZWkxDUkZUeVZERXJXQ2VGVzRXTlNpN0JXeFhMblJZVWgz?=
 =?utf-8?B?USt5K0s0SWZnMGs5a0pSQlh4Nk1EOHNybUZtYUp2QnpUanRpdUtKTVBKZnVM?=
 =?utf-8?B?TUp6dXlQZDNEMjZQOUdldFZZVlV5N2dJTklZbG8xK0h0ZHlvbTN4NVFoQ3cr?=
 =?utf-8?B?eEFYZllLQk5KRWM4MC9zYVBzYk9rdnpaUFljZzVqK2hJaWdreEZSTEVrQzJU?=
 =?utf-8?B?VDUyMG5PQkhBdWFjQXkraks0SUh2czVoa2RVS3ozQ0poZmtqZnNwU1R2a2tM?=
 =?utf-8?Q?q2KkunDjCLPuOOEGg6dq0zleD?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 57013323-7cab-4be2-80c7-08db1f11b633
X-MS-Exchange-CrossTenant-AuthSource: BL1PR12MB5874.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Mar 2023 13:41:53.6353
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: VFFhvKRsDkbrPeHjK0SqQQlciseerfIjyvqJsAEkm7K9DNVhFCd+MBOPXr/LkzfLyuoZkiVqvgBmRHzO6n3N0g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR12MB8244
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 3/6/23 17:03, Akira Yokosawa wrote:
> On Mon, 6 Mar 2023 09:34:29 -0600, Carlos Bilbao wrote:
>> On 3/6/23 09:30, Akira Yokosawa wrote:
>>> On 2023/03/07 0:20, Carlos Bilbao wrote:
>>>> Hello Akira,
>>>>
>>>> On 3/6/23 09:13, Akira Yokosawa wrote:
>>>>> Hi Carlos,
>>>>>
>>>>> Minor nits in the Subject and Sob area.
>>>>>
>>>>> On Mon, 6 Mar 2023 07:44:20 -0600, Carlos Bilbao wrote:
>>>>>> Subject: [PATCH] docs/sp_SP: Add process deprecated translation
>>>>>
>>>>> This summary looks ambiguous to me.
>>>>>
>>>>> Maybe
>>>>>
>>>>>       docs/sp_SP: Add translation of process/deprecated
>>>>
>>>> This summary follows the same format followed in the past. Some examples:
>>>>
>>>> docs/sp_SP: Add process coding-style translation
>>>> docs/sp_SP: Add process magic-number translation
>>>> docs/sp_SP: Add process programming-language translation
>>>> docs/sp_SP: Add process email-clients translation
>>>
>>> Let me explain why "Add process deprecated translation" looks
>>> ambiguous.
>>>
>>> "deprecated translation" can be interpreted as "some translation
>>> which is deprecated".
>>> Of course you don't need to agree.
>>
>> I see what you mean. I'm sending v2 patch renamed to avoid confusion.
>>
>>>
>>>>
>>>>>
>>>>> ??
>>>>>
>>>>>> Translate Documentation/process/deprecated.rst into Spanish.
>>>>>>
>>>>>> Co-developed-by: Carlos Bilbao <carlos.bilbao@amd.com>
>>>>>> Signed-off-by: Sergio Gonzalez <sergio.collado@gmail.com>
>>>>>> Signed-off-by: Carlos Bilbao <carlos.bilbao@amd.com>
>>>>>
>>>>> To me, Co-developed-by: from the author of the patch looks
>>>>> strange, because it is obvious the author did some development on
>>>>> the patch.
>>>>>
>>>>
>>>> No, we both worked on this patch so Co-developed-by: is the appropriate
>>>> tagging. That being said, Sergio translated more than I did, so I put
>>>> him as sole Translator in the document itself.
>>>
>>> Hmm, anyway I don't think you are following the rule of Co-developed-by:
>>> explained in submitting-patches.rst.
>>>
>>> Again, you don't need to agree... ;-)
>>
>> But, why doesn't it follow the rule?
>>
>> The rule is "A Co-Developed-by: states that the patch was also created by another developer along with the original author. This is useful at times when multiple people work on a single patch."
>>
>> IMHO this is the case here, but before I send v2 I'll wait to read you again in case we agree at that point.
> 
> If you put "From: Sergio" as the first line in the Changelog, like
> this submission [1], then the Sob chain would make sense.
> 
> [1]: https://lore.kernel.org/linux-doc/20230227222957.24501-2-rick.p.edgecombe@intel.com/
> 
> Didn't you forgot to put it there?

Sending v2 :)

> 
> Just guessing...
> 
>          Thanks, Akira
> 
>>
>>>
>>>           Thanks, Akira
>>>
>>>>
>>>>> Which is your intent:
>>>>>
>>>>>       Author: Carlos
>>>>>       Co-developer: Sergio
>>>>>
>>>>> , or
>>>>>
>>>>>       Author: Sergio
>>>>>       Co-developer: Carlos
>>>>>
>>>>> ???
>>>>>
>>>>>            Thanks, Akira
>>>>>
>>>>>> ---
>>>>>>     .../translations/sp_SP/process/deprecated.rst | 381 ++++++++++++++++++
>>>>>>     .../translations/sp_SP/process/index.rst      |   1 +
>>>>>>     2 files changed, 382 insertions(+)
>>>>>>     create mode 100644 Documentation/translations/sp_SP/process/deprecated.rst
>>>>> [...]
>>>>
>>>> Thanks,
>>>> Carlos
>>
>> Thanks,
>> Carlos

Thanks,
Carlos