Re: [PATCH v3] Documentation: cve Korean translation

[Yunseong Kim]

On 5/27/24 10:50 오후, Jonathan Corbet wrote:
> Yunseong Kim <yskelg@xxxxxxxxx> writes:
> 
>>> 1) Why do I have three versions of it in my mailbox, sent over a period
>>>    of 13 minutes?  What changed between the versions?
>>
>> Sorry, I forgot the name of the reviewer when I first sent the
>> documentation content related patch version 2.
> 
> Which is fine, but...
> 
>>>    Normally, you want to wait for reviews to come in on one version
>>>    before posting the next, and you should put a comment after the "---"
>>>    line saying what changed.
>>>
>>> 2) When did this review from Jinwoo Park happen?  I was not copied on
>>>    that.
> 
> You did not answer this question.  Reviews should generally be done in
> public, but that does not seem to have happened here?

Oops, sorry about that, Jonathan.

Jinwoo Park sent me the review below, and I've updated some of ambiguous
words in patch version 2.

https://lore.kernel.org/linux-doc/57f0d90c-4cc6-4418-ab79-6ae026d8ae09@xxxxxxxxx/T/#t

>> Thanks for translating new Documentation/process/cve document to Korean
>> Language. Most of the Korean sentences are looks good to me. But only
>> one sentence seemed unnatural.
>
> Thank you for the review Jinwoo.
>
>>> 잘못된 CVE 항목들
>>> =================
>>>
>>> -해당 배포판에서 변경된 사항으로 인해 또는 해당 배포판이 더 이상
>>> kernel.org
>>> +특정 배포판에서 변경된 사항으로 인해 또는 해당 배포판이 더 이상
>>> kernel.org
>>> 지원 릴리스가 아닌 커널 버전을 지원하기 때문에 Linux 배포판에서만
지원되는
>>> Linux 커널에서 보안 문제가 발견되는 경우 Linux 커널 CVE 팀에서 CVE를
할당할
>>> 수 없으며 해당 Linux 배포판 자체에서 요청해야 합니다.
>>
>> When the first modifier "해당" is first used in a Korean sentence, like
>> "the", there needs to be an explanation of what it is targeting.
>
>
> You're right, that phrase was awkward in the direct translation. Thanks
> for catching that.
>
>> However, in the process of literal translation, it seems that "the"
>> became "해당" due to the difference in word order between Korean and
>> English, And since the translated sentence did not describe which "Linux
>> distributor" is being described, it would be very difficult if
>> "특정(specific)" were used instead. It seems natural.
>
> I will send version 2 patch. Thank you!
>
> Warm Regards,
>
> Yunseong Kim

And I added Jinwoo as a reviewer to the documentation on patch version 3.

On 5/27/24 10:21 오후, Yunseong Kim wrote:
> I added Jinwoo's name here on version 3.
>
>> +:감수: 박진우 <pmnxis@xxxxxxxxx>
> Warm Regards,

Thank you for guiding me Jonathan.

I'll keep the process in mind and use them to help the next person who
translates to Korean.

> Thanks,
> 
> jon


Warn Regards,

Yunseong Kim