On Tue, Apr 12, 2022 at 12:55:39PM +0800, Tiezhu Yang wrote: > > > On 04/12/2022 11:50 AM, Darrick J. Wong wrote: > > On Mon, Apr 11, 2022 at 08:39:17PM -0700, Darrick J. Wong wrote: > > > On Tue, Apr 12, 2022 at 11:21:26AM +0800, Tiezhu Yang wrote: > > > > Remove the following section entries of IOMAP FILESYSTEM LIBRARY: > > > > > > > > M: linux-xfs@xxxxxxxxxxxxxxx > > > > M: linux-fsdevel@xxxxxxxxxxxxxxx > > > > > > > > Remove the following section entry of XFS FILESYSTEM: > > > > > > > > M: linux-xfs@xxxxxxxxxxxxxxx > > > > > > > > Signed-off-by: Tiezhu Yang <yangtiezhu@xxxxxxxxxxx> > > > > > > WTF? > > > > > > ▄▄ ▄ ▄▄ ▄ ▄ > > > █▀▄ █ ██ █ ▄▀ > > > █ █▄ █ █ █ █▄█ > > > █ █ █ █▄▄█ █ █▄ > > > █ ██ █ █ █ ▀▄ > > > > *OH*, I see, you're getting rid of the M(ail): entry, probably because > > it's redundant with L(ist): or something?? Still... why does it matter? > > Yes, the section entries are redundant. Sorry for the unclear description. > > The intention of this patch is to clean up the redundant section entries. > > > > > Seriously, changelogs need to say /why/ they're changing something, not > > simply restate what's already in the diff. > > OK, thank you. Should I send a v2 patch to update the commit message > or just ignore this patch? Yes, please send a v2 patch with an improved commit message. NAK withdrawn. --D > Thanks, > Tiezhu >