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. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot