<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 24, 2016 at 2:53 PM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="">On Tue, 24 May 2016, Mark Adams wrote:<br>
<br>
> Do I now wait for my pull request to be approved?<br>
<br>
</span>Mark,<br>
<br>
Currently there is a bit of mess in regards to your branches.<br></blockquote><div><br></div><div>Yes. This got messy because I was building on Matt's work.  mark/snes-ex56c super seeds everything else.</div><div><br></div><div>I used cherry-pick to duplicate commits in next. This may explain the missing merges.</div><div><br></div><div>I checked Barry's commits and saw the mark/snes-ex56b and picked it in mark/snes-ex56c.</div><div><br></div><div>I will clean up these old ex56's</div><div><br></div><div>Let me know if I missed anything below.</div><div><br></div><div>Thanks,</div><div>Mark</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<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>
<span class=""><br>
$ git branch -r --contains eb041292<br>
  origin/mark/snes-ex56c<br>
  origin/next<br>
<br>
</span>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 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 'pr476/psanan/psanan/pc-telescope-subcomm-type/master' into next<br>
83661ee Merge branch '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>
<span class="">a003960 Merge branch 'master' of bitbucket.org:petsc/petsc into mark/snes-ex56c<br>
<br>
</span>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>
</blockquote></div><br></div></div>