From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: linux-next: Signed-off-by missing for commits in the arm-soc tree Date: Mon, 16 Jul 2018 10:57:00 +0100 Message-ID: <20180716095700.GC17271@n2100.armlinux.org.uk> References: <20180716075718.4e5ef491@canb.auug.org.au> <6a9fa6f3-7aef-a83b-72a6-dc9492964b5e@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Chen-Yu Tsai Cc: Florian Fainelli , Stephen Rothwell , Arnd Bergmann , Linux Kernel Mailing List , Eric Anholt , Linux-Next Mailing List , Olof Johansson , ARM List-Id: linux-next.vger.kernel.org On Mon, Jul 16, 2018 at 05:24:08PM +0800, Chen-Yu Tsai wrote: > On Mon, Jul 16, 2018 at 5:13 PM, Florian Fainelli wrote: > > > > > > On 07/15/2018 03:50 PM, Olof Johansson wrote: > >> Thanks Stephen, I keep saying every time you catch these that I need > >> to run the same script. :( > >> > >> Florian, I wonder if this happened when you rebased to squash in the fix? > > > > Humm, could be, all I did (and it was not the first time) was to do an > > interactive rebase with --preserve-merges. AFAICT, all of these commits > > came from Eric's pull request, so what I typically do is just sign off > > on the merge commit, but do not apply my SoB to all commits coming from > > that pull request if that makes sense? > > When you rebase, if the commit is re-applied, the committer changes to > you, so you need to add your SoB to all commits you rebased. > > I suppose you could override it with GIT_COMMITTER_NAME and > GIT_COMMITTER_EMAIL environment variables, but that sort of distorts > the SoB if any changes were added due to rebasing, such as resolution > of merge conflicts. A different point: should *anyone* be rebasing commits that they did not themselves commit. Discuss. IMHO no - the commits were made public, and anyone could pull the tree from which they came in order to do further work before submitting that. Rebasing changes the commit IDs, which can lead to duplicate commits ending up in mainline. With other changes on top, this causes totally unnecessary conflicts. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 13.8Mbps down 630kbps up According to speedtest.net: 13Mbps down 490kbps up