Re: [PATCH] lib/Kconfig.debug: fix grammar in RUST_BUILD_ASSERT_ALLOW

[Timo Grautstück]

Miguel Ojeda <miguel.ojeda.sandonis@xxxxxxxxx> writes:

> On Mon, Jun 5, 2023 at 6:12 PM <timo.gr@xxxxxxxxxx> wrote:
>>
>> From: Timgrau <timo.gr@xxxxxxxxxx>
>>
>> Just a grammar fix in lib/Kconfig.debug, under the config option RUST_BUILD_ASSERT_ALLOW.
>>
>> Signed-off-by: Timgrau <timo.gr@xxxxxxxxxx>
>> Suggested-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
>> Link: https://github.com/Rust-for-Linux/linux/issues/1006#issue-1696318406
>> Closes: https://github.com/Rust-for-Linux/linux/issues/1006#issue-1696318406
>
> Thanks for the patch!
>
> A few notes:
>
>   - From a quick look, I think "Timgrau" is not a known identity to
> the kernel, right? (e.g. I couldn't find any other commit on that
> name). Assuming I am not mistaken, could you please instead your real
> name?

Right, I already thought so. My bad, will change it.
>
>   - When I wrote the report, I wrongly suggested `Suggested-by` +
> `Link` because I didn't consider this a "bug". But you are probably
> right that should be considered a fix anyway, so instead we should use
> `Reported-by` and `Closes`. So could you please remove the `Link` and
> change the `Suggested` to `Reported`?

Ahh ok, i also dont think you can call it a "bug". Is it not possible to
simply remove the `Closes:` tag from the patch and not categorize the
whole thing as a "bug"?
>
>   - Also, since this is now a fix, could you please also add the `Fixes:` tag?
>
>   - For the `Closes`, you can remove the URL fragment part, i.e.
> `#issue-1696318406`, since it is anyway the first comment.

In that case, these two points/notes above are not needet right?
>
> If you could change those bits and send a v2, that would be great. Thanks!
>
> Cheers,
> Miguel