<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
</span>Firstly - we should not be cherry-picking commits to next. Its an<br>
'integration branch' i.e any required fixes go into the feature branch<br>
that might have triggered the issue.<br></blockquote><div><br></div><div>So I should just (re)merge my branch with my local next, as I fix it, pull from next, push to next.</div><div><br></div><div>I am thinking what caused problems was that I was getting Matt's fixes in next.  I should have merged by branch with Matt's branch(es), fixed things, then merge with next, pull/push next to origin.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Secondly - as explained - the merge commit exists. Its text is<br>
modified - and a change commit was also included in this merge<br>
commit. This should be avoided. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> I checked Barry's commits and saw the mark/snes-ex56b and picked it in<br>
> mark/snes-ex56c.<br>
<br>
</span>Ok - this suggests I should revert 'mark/snes-ex56b' merge from next.<br>
But there are other things in next that shouldn't be there [and that<br>
makes it difficult to track where the breakages in next are coming<br>
from..<br></blockquote><div><br></div><div>Sorry,</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
Satish<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> I will clean up these old ex56's<br>
><br>
> Let me know if I missed anything below.<br>
><br>
> Thanks,<br>
> Mark<br>
><br>
><br>
> ><br>
> > $ git branch -r |grep mark |grep ex56<br>
> >   origin/mark/ksp-ex56<br>
> >   origin/mark/ksp-newex56<br>
> >   origin/mark/snes-ex56b<br>
> >   origin/mark/snes-ex56c<br>
> ><br>
> > You have a bunch of branches with the name 'ex56' - presumably you<br>
> > worked initially on mark/snes-ex56b and switched to 'mark/snes-ex56c'.<br>
> ><br>
> > And I see the the following 2 merges to next<br>
> ><br>
> > >>>>>>><br>
> > commit e223b1660d8274c4632642ec520eee8c34d18eb5<br>
> > Merge: de2cc15 c13d8fd<br>
> > Author: Barry Smith <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>><br>
> > Date:   Mon May 23 15:27:49 2016 -0500<br>
> ><br>
> >     Merge branch 'mark/snes-ex56b' into next<br>
> ><br>
> > commit ebff70bc705bbe7fea86844e10d9a274abdf3658<br>
> > Merge: c8d3fd4 dfc6d4a<br>
> > Author: Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>><br>
> > Date:   Thu May 19 23:59:01 2016 -0500<br>
> ><br>
> >     Merge branch 'mark/snes-ex56c' into next<br>
> > <<<<<<<<br>
> ><br>
> > On my part - I noticed you had a buggy commit in next - which I wanted<br>
> > to fix - so fixed in 'mark/snes-ex56c' and merged to next.<br>
> ><br>
> > $ git branch -r --contains eb041292<br>
> >   origin/mark/snes-ex56c<br>
> >   origin/next<br>
> ><br>
> > But how did this commit come to next? I don't see a prior 'Merge<br>
> > branch 'mark/snes-ex56c' message in next history. Ok I think<br>
> > c8d3fd49b4313a86346f23ef7d5371efba2d3dc9 was the next HEAD - when I<br>
> > did the merge.<br>
> ><br>
> > $ git log --oneline --merges --ancestry-path<br>
> > eb041292..c8d3fd49b4313a86346f23ef7d5371efba2d3dc9<br>
> > c8d3fd4 Merge branch 'next' of bitbucket.org:petsc/petsc into next<br>
> > 01528be Merge branch 'master' into next<br>
> > 8bd3767 Merge branch<br>
> > 'pr476/psanan/psanan/pc-telescope-subcomm-type/master' into next<br>
> > 83661ee Merge branch<br>
> > 'pr476/psanan/psanan/pc-telescope-subcomm-type/master' into next<br>
> > 2a9b9f0 Merge branch 'master' into next<br>
> > d5742a2 fixed error check in main of ex56<br>
> > a003960 Merge branch 'master' of bitbucket.org:petsc/petsc into<br>
> > mark/snes-ex56c<br>
> ><br>
> > So 'd5742a2 fixed error check in main of ex56' is a merge commit?<br>
> ><br>
> > You did code change and merge in the same commit - and removed the<br>
> > default git commit message - "Merge 'mark/snes-ex56c' into next"?<br>
> ><br>
> > This is way confusing..<br>
> ><br>
> > Now to figure out how to fix this. I see origin/mark/snes-ex56b is<br>
> > merged to next - but origin/mark/snes-ex56c is not completely merged.<br>
> ><br>
> > Is origin/mark/snes-ex56b abandoned? If so - I should revert Barry's<br>
> > inadvertent merge of origin/mark/snes-ex56b into next - and merge<br>
> > origin/mark/snes-ex56c<br>
> ><br>
> > Thanks,<br>
> > Satish<br>
> ><br>
><br>
<br>
</div></div></blockquote></div><br></div></div>