linux-kernel - Re: [GIT PULL] Please pull powerpc/linux.git powerpc-4.6-1 tag

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1458552497.7374.1.camel@ellerman.id.au>
Date:	Mon, 21 Mar 2016 20:28:17 +1100
From:	Michael Ellerman <mpe@...erman.id.au>
To:	Linus Torvalds <torvalds@...ux-foundation.org>
Cc:	Igal.Liberman@...escale.com, Zhiqiang.Hou@...escale.com,
	adam.buchbinder@...il.com, aik@...abs.ru,
	alessio.bogani@...ttra.eu, andrew.donnellan@....ibm.com,
	"Aneesh Kumar K. V" <aneesh.kumar@...ux.vnet.ibm.com>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
	Bjorn Helgaas <bhelgaas@...gle.com>,
	Boqun Feng <boqun.feng@...il.com>,
	Balbir Singh <bsingharora@...il.com>,
	chenhui.zhao@...escale.com,
	Christophe Leroy <christophe.leroy@....fr>,
	clombard@...ux.vnet.ibm.com, cyrilbur@...il.com,
	david@...son.dropbear.id.au, dongsheng.wang@...escale.com,
	dougmill@...ux.vnet.ibm.com, Torsten Duwe <duwe@....de>,
	duwe@...e.de, fbarrat@...ux.vnet.ibm.com,
	gwshan@...ux.vnet.ibm.com, Ian Munsie <imunsie@....ibm.com>,
	Jeremy Kerr <jk@...abs.org>, joel@....id.au,
	"Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
	leoyang.li@....com,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	ppc-dev <linuxppc-dev@...ts.ozlabs.org>,
	luis.henriques@...onical.com, maddy@...ux.vnet.ibm.com,
	manoj@...ux.vnet.ibm.com, Michael Neuling <mikey@...ling.org>,
	mrochs@...ux.vnet.ibm.com, oss@...error.net,
	Paul Mackerras <paulus@...abs.org>,
	Paul Mackerras <paulus@...ba.org>, qiang.zhao@....com,
	raghav.dogra@....com, ruscur@...sell.cc, saurabh.truth@...il.com,
	sjitindarsingh@...il.com, stewart@...ux.vnet.ibm.com,
	sukadev@...ux.vnet.ibm.com, vaibhav@...ux.vnet.ibm.com,
	Richard Yang <weiyang@...ux.vnet.ibm.com>,
	xinhui.pan@...ux.vnet.ibm.com, xuelin.shi@....com
Subject: Re: [GIT PULL] Please pull powerpc/linux.git powerpc-4.6-1 tag

On Sat, 2016-03-19 at 15:57 -0700, Linus Torvalds wrote:
> On Fri, Mar 18, 2016 at 5:07 AM, Michael Ellerman <mpe@...erman.id.au> wrote:
> >
> > I couldn't convince git request-pull to generate a sane diffstat below, it
> > seems to be confused because I merged powerpc-4.5-4 into my next.
>
> Yes, if there are multiple merge bases (particularly cross-merges, but
> you can get it just from internal merges of work that started at
> different points), a plain "git diff" will often be unable to give
> reasonable results.
>
> > So I just did the merge and took that diffstat. Hopefully that makes sense.
>
> Yes, that's the way to get a reliable diff if you have more complex history.

OK.

For some reason I thought git-request-pull had some magic to work out the right
diffstat, but I see now it's just a shell script that does git diff --stat.

> Some people do that merge by default (ie both the networking tree and
> the ARM SoC trees tend to do this), not just to get the reliable diff,
> but also to inform me of the conflicts that they encountered.

Yeah, I have been doing it to check for conflicts, but then throwing it away.
So I'll just change my process to do the test merge and then take the diff stat
for the pull request.

> Basically you should never hit that complicated case unless you are
> already using workflows that means that you can handle it, and by then
> you usually also end up understanding why clean history and avoiding
> back-merges is a good idea etc.
>
> Welcome to that second group.

Thanks.

cheers