Conversation
@kernellogger I see the "Reported-by: should be immediately followed by Closes: with a URL to the report" stuff got merged. I really dislike that it whines if I do Reported-by: followed by Link:.
1
0
0
@kernellogger @matttbe yeah, partially due to patches that only fix part of a reported issue (I think I was the one who mentioned it on LKML) but also "Closes:" is a very definitive statement, when quite often a fix does not in fact entirely fix the problem.
1
0
0
@kernellogger @matttbe I would like to make my CI stuff entirely ignore that tag, complaint, but I don't immediately see an ignore.
0
0
0
@matttbe @kernellogger checkpatch isn't an arbiter of truth to begin with, especially in arch code it reports a bunch of spurious stuff.

I was just a bit annoyed by it the other day when checkpatch complained, telling me that I should have a Closes: pointing to a competing patch for a problem.
I guess I had had it in my head that what was accepted was going to allow Reported-by: followed by Link: & complain only about "bare" Reported-by:.

> it is similar to the previous situation where there are cases where it doesn't even make sense to have a Link tag after a Reported-by.

Yah, that's probably fair.
0
0
1