[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH 1/7] xz: add fall-through comments to a switch statement
On 26.11.2021 11:04, Julien Grall wrote: > Hi Jan, > > On 26/11/2021 09:12, Jan Beulich wrote: >>> Anyway, I think it would save time for everyone (you had to manually >>> delete signed-off-by after all) if you just copy the commit (including >>> all the signed-off-by) message as-is. >> >> I don't think I see why you found it necessary to verify the S-o-b set. > > This is a list of difference with the Linux commit that was unexplained > to me. > >> >> Also note that, for things to be useful in our tree, I may also edit >> commit messages in mechanical ways (e.g. to change file or function >> names). I don't think you can expect a 1:1 match in any event. > > I am fully aware that I can't expect a 1:1 match. However, if I see a > difference, then I need to be able to explain it. > > For this case, you provided some sort of an explanation but so far, I am > still waiting for a link to confirm that the signed-off-by match the one > on the ML. I haven't been able to easily find a mail archive holding this patch. However, to me http://lkml.iu.edu/hypermail/linux/kernel/1710.1/04375.html clearly suggests that Jiri merely took the patch and applied it. For patches 2 and onwards the Linux commits contain links (which I did also retain in the posted patches), i.e. https://lore.kernel.org/all/20191104185107.3b6330df@xxxxxxxxxxx/T/#u https://lore.kernel.org/all/20210604085656.12257-1-thunder.leizhen@xxxxxxxxxx/T/#u https://lore.kernel.org/all/20211010213145.17462-2-xiang@xxxxxxxxxx/ https://lore.kernel.org/all/20211010213145.17462-6-xiang@xxxxxxxxxx/ https://lore.kernel.org/all/20211010213145.17462-4-xiang@xxxxxxxxxx/ https://lore.kernel.org/all/20211010213145.17462-3-xiang@xxxxxxxxxx/ Going through those made me notice that on patches 4 and onwards I should put back one more S-o-b, albeit for all four I then can't help thinking that authorship is really the other way around. But I'm not going to put effort into finding out ... This exercise also made me notice that I have the last three patches the wrong way round. Not that this would matter much. FTAOD: Are these further inquiries of yours actually intended to tell me that I should not have applied the ack that you've sent first for this one patch? Jan
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |