Re:Re: [PATCH] [v2] docs/zh_CN: add translations in zh_CN/dev-tools/kasan
From: Jiabing Wan
Date: Thu Jun 03 2021 - 00:50:19 EST
> Re: [PATCH] [v2] docs/zh_CN: add translations in zh_CN/dev-tools/kasan>>Hi Jiabing,
>
>Thanks for translation! The contents are generally fine.
>I tried the patch with scripts/checkpatch.pl found there are lots errors
Hi,Alex,
Oh, that is strange.
In my ubuntu server, I ran scripts/checkpatch.pl and got:
When Documentation/translations/zh_CN/dev-tools/kasan.rst
"total: 0 errors, 0 warnings, 431 lines checked"
When 0001-v2-docs-zh_CN-add-translations-in-zh_CN-dev-tools-ka.patch.
"WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#41:
new file mode 100644
total: 0 errors, 1 warnings, 445 lines checked"
No errors and just one warning.
So I think there was a problem with your platform or downloading?
If there were no problem, I would check my email client again ;)
>---
>...
>ERROR: DOS line endings
>#552: FILE: Documentation/translations/zh_CN/dev-tools/kasan.rst:430:
>+^M$
>
>ERROR: trailing whitespace
>#553: FILE: Documentation/translations/zh_CN/dev-tools/kasan.rst:431:
>+.. _KUnit: https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html^M$
>
>total: 432 errors, 1 warnings, 445 lines checked
>---
>
>and the lines in the patch are not aligned on 80 chars, (40 chars for Chinese,
>since one Chinese char could take 2 positions. )
>Thanks
>Alex
OK, great tips for me.
I'll fix them in v3.
Thanks,
Jiabing
>
>On 2021/6/2 下午8:56, Wan Jiabing wrote:
>> Add new zh translations
>> * zh_CN/dev-tools/kasan.rst
>> and link it to zh_CN/dev-tools/index.rst
>>
>> Signed-off-by: Wan Jiabing <wanjiabing@xxxxxxxx>
>> ---
>> Changelog:
>> v2:
>> - Delete spaces surround with English words.
>> ---