From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Sat, 9 Nov 2013 18:49:01 +0100 Subject: [U-Boot] Non-trivial merge conflicts between u-boot/master and u-boot-arm/master In-Reply-To: <20131109164735.GK5925@bill-the-cat> References: <20131109141115.0ce751ab@lilith> <20131109164735.GK5925@bill-the-cat> Message-ID: <20131109184901.42347c29@lilith> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tom, On Sat, 9 Nov 2013 11:47:35 -0500, Tom Rini wrote: > On Sat, Nov 09, 2013 at 02:11:15PM +0100, Albert ARIBAUD wrote: > > Hello, > > > > While preparing my pull request from ARM to mainline, I've tried a > > merge of u-boot-arm/master and u-boot/master and had to solve a few > > non-trivial conflicts due to the kbuild stuff. > > > > I wonder how I should proceed now. Should I simply submit the PR and > > warn Tom that conflicts will arise, and indicate how I solved them? > > I'd do (and I've done) that for trivial cases, but for non-trivial > > changes it seems error-prone. > > > > Plus, I have already performed the resolutions, so why waster Tom's > > time? I could forward u-boot-arm/master to the merge commit, then > > submit a fast-forward PR to u-boot/master. > > > > But then, the changes I did will remain un-reviewed or maybe even > > unnoticed. > > > > I also thought I could treat this as a normal patch and submit it to the > > ML... Only git won't generate a "patch" for merge commits, and I don't > > know how patchwork will react to this. > > > > So... any advice? > > Include the resolution in the PR, and include the not-a-diff-exactly > that git will generate, include that in the PR. Which 'not-a-diff-exactly' do you mean? Amicalement, -- Albert.