4 ms·
He mentioned it in his PR summary as a new option. About half of the summary of the original PR was talking about the new option and why it was important. http
by NewJazz 1y ago
He mentioned it in his PR summary as a new option. About half of the summary of the original PR was talking about the new option and why it was important.
https://lore.kernel.org/linux-fsdevel/4xkggoquxqprvphz2hwnir7nnuygeybf2xzpr5a4qtj4cko6fk@dlrov4usdlzm/ https://lore.kernel.org/linux-fsdevel/4xkggoquxqprvphz2hwnir...
- ajb 1y agoI'm not saying he made a PR just saying "Fixes" like a rookie. What I'm saying is that in there should have been something along the lines of "heads up - I know this doesn't comply with the usual process for the following commits, here's why I think they should be given a waiver under these circumstances" followed by the justifications that appeared after Linus got upset. The PR description would have been fine - if it had been in the right stage of the process.