From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: linux-next: Signed-off-by missing for commits in the arm-soc tree Date: Mon, 16 Jul 2018 04:55:50 -0700 Message-ID: <6a6d47fa-cc7f-b977-339e-ea215bc40541@gmail.com> References: <20180716075718.4e5ef491@canb.auug.org.au> <6a9fa6f3-7aef-a83b-72a6-dc9492964b5e@gmail.com> <20180716095700.GC17271@n2100.armlinux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180716095700.GC17271@n2100.armlinux.org.uk> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Russell King - ARM Linux , Chen-Yu Tsai Cc: 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 07/16/2018 02:57 AM, Russell King - ARM Linux wrote: > 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. > Well in that case, I thought it would be cleaner to just squash Arnd's build fix since the commit had not been merged into arm-soc, but I will keep in mind not to do that anymore. -- Florian