[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ACYAbADLDo5nvv4af7zOLqqA.3.1622695812793.Hmail.wanjiabing@vivo.com>
Date: Thu, 3 Jun 2021 12:50:12 +0800 (GMT+08:00)
From: Jiabing Wan <wanjiabing@...o.com>
To: Alex Shi <seakeel@...il.com>, Alex Shi <alexs@...nel.org>,
Jonathan Corbet <corbet@....net>,
Paul Walmsley <paul.walmsley@...ive.com>,
Palmer Dabbelt <palmer@...belt.com>,
Albert Ou <aou@...s.berkeley.edu>,
Nathan Chancellor <nathan@...nel.org>,
Nick Desaulniers <ndesaulniers@...gle.com>,
Wu XiangCheng <bobwxc@...il.cn>,
Fangrui Song <maskray@...gle.com>,
Bernard Zhao <bernard@...o.com>, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-riscv@...ts.infradead.org,
clang-built-linux@...glegroups.com
Subject: Re:Re: [PATCH] [v2] docs/zh_CN: add translations in zh_CN/dev-tools/kasan
> 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@...o.com>
>> ---
>> Changelog:
>> v2:
>> - Delete spaces surround with English words.
>> ---
Powered by blists - more mailing lists