Re: [PATCH] docs/sp_SP: Add translation of process/adding-syscalls

[Carlos Bilbao <carlos.bilbao@amd.com>]

On 5/2/23 12:16, Jonathan Corbet wrote:
> Carlos Bilbao <carlos.bilbao@amd.com> writes:
> 
>> Hello Jon,
>>
>> I applied on top of my most recent docs-next version:
>>
>> 0b656310bfc33c5e044df9cf6013df8280ec69cc ("docs/admin-guide/mm/ksm.rst fix
>> intraface -> interface typo")
>>
>> and it worked, and make htmldocs too. Could you please tell me what errors
>> or warnings are you experiencing?
> 
> How did you try to apply it?  I get the dreaded:
> 
>    Applying: docs/sp_SP: Add translation of process/adding-syscalls
>    error: patch failed: Documentation/translations/sp_SP/process/index.rst:18
>    error: Documentation/translations/sp_SP/process/index.rst: patch does not apply
>    Patch failed at 0001 docs/sp_SP: Add translation of process/adding-syscalls
> 
> The problem was this hunk:
> 
> diff --git a/Documentation/translations/sp_SP/process/index.rst
> b/Documentation/translations/sp_SP/process/index.rst
> index 351bcd3921ba..a0ff2e132c54 100644
> --- a/Documentation/translations/sp_SP/process/index.rst
> +++ b/Documentation/translations/sp_SP/process/index.rst
> @@ -18,3 +18,4 @@
>      email-clients
>      programming-language
>      deprecated
> +   adding-syscalls
> 
> I have your addition of magic-number.rst applied, you seemingly did not
> when the patch was generated.
> 
> Figuring these things out takes time.  Next time something like this
> happens, can you just regenerate the patch against the current tree and
> resend?

I definitely will, thank you Jon.

> 
> Anyway, I've applied it.
> 
> Thanks,
> 
> jon

Best,
Carlos