From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Date: Sat, 03 Oct 2020 19:36:10 +0000 Subject: Re: [PATCH 00/18] use semicolons rather than commas to separate statements Message-Id: <306974234222092651f7497504f8c03e1304926b.camel@perches.com> List-Id: References: <1601233948-11629-1-git-send-email-Julia.Lawall@inria.fr> <160132172369.55460.9237357219623604216.b4-ty@kernel.org> <20200929113745.GB4799@sirena.org.uk> <20201001110150.GA6715@sirena.org.uk> <20201003191501.o56tqq63d2buq5ox@chatter.i7.local> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Konstantin Ryabitsev Cc: Mark Brown , tools@linux.kernel.org, linux-iio@vger.kernel.org, Julia Lawall , linux-stm32@st-md-mailman.stormreply.com, linux-crypto@vger.kernel.org, "Rafael J. Wysocki" , linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, Jerome Brunet , linux-acpi@vger.kernel.org, David Lechner , Valdis =?UTF-8?Q?Kl=C4=93tnieks?= , kernel-janitors@vger.kernel.org, drbd-dev@lists.linbit.com, openipmi-developer@lists.sourceforge.net, Martin Blumenstingl , linux-ide@vger.kernel.org, linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Thomas Gleixner , linux-wireless@vger.kernel.org, Neil Armstrong On Sat, 2020-10-03 at 12:27 -0700, Joe Perches wrote: > On Sat, 2020-10-03 at 15:15 -0400, Konstantin Ryabitsev wrote: > > On Sat, Oct 03, 2020 at 11:40:48AM -0700, Joe Perches wrote: > > > (Adding tools and Konstantin Ryabitsev) > > > > > > There seems to be some mismatch between b4's use of the > > > cover letter to a patch series and what maintainers that > > > apply a subset of the patches in the patch series. > > > > > > The merge description shows the entire patch series as > > > applied, but the actual merge is only a subset of the > > > series. > > > > > > Can this be improved in b4? > > > > So, the following logic should be applied: > > > > - if the entire series was applied, reply to 0/n > > - if a subset only is applied, reply to each n/n of the patch that was > > cherry-picked out of the series > > > > Is that an accurate summary? > > Exactly so, thanks. And there's no need to commit the [0/n] cover letter as a part of the merge unless the entire series was committed. Or perhaps trim the cover letter to exclude the files modified by the patch series and show only the actual files committed. And I believe b4 inserts this line ahead of the 0/n series cover letter description for the merge: Merge series "" from : Perhaps that like could be "partial merge of" when a partial merge occurs or left as is if the entire series is applied. cheers, Joe