<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Merged into master:</span><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">
<a href="https://bitbucket.org/petsc/petsc/commits/d27186240524b7073c7cded90e339bd6b28bd6e6" target="_blank">https://bitbucket.org/petsc/petsc/commits/d27186240524b7073c7cded90e339bd6b28bd6e6</a></div><div style="font-family:arial,sans-serif;font-size:13px">
<br></div><div style="font-family:arial,sans-serif;font-size:13px">- Peter</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 2:04 PM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> writes:<br>

<br>
> On Sat, 2 Nov 2013, Jed Brown wrote:<br>
><br>
>> Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> writes:<br>
>> > Should 'prbrune/sf-sfbasicops' and 'prbrune/mat-matcolor' be cleaned<br>
>> > up to remove the merge between the 'rebased' and 'non-rebased'<br>
>> > variants of the branches - before the changes got into master?<br>
>><br>
>> This looks possible.<br>
>><br>
>> *   5c44807 (prbrune/mat-matcolor) Merge branch 'prbrune/sf-sfbasicops' into prbrune/mat-matcolor<br>
>> |\<br>
>> | *   8e99f7e (prbrune/sf-sfbasicops) Merge branch 'prbrune/sf-sfbasicops' of bitbucket.org:petsc/petsc into prbrune/sf-sfbasicops<br>
>> | |\<br>
>> | | * f3fe83a mpiuni: change types to use unique IDs [for sf]<br>
>> | | * 13d1d6f mpiuni: add MPI_OP definitions required for sfbasic<br>
>> | * | a49f225 Added several additional MPI_Ops to PetscSFBasic.<br>
>> |  /<br>
>> * | 3142415 Fix for multiple definitions in mat/color/interface/ custom fortran bindings<br>
>><br>
>><br>
>> $ diff -u  <(git show --format=fuller 53deab3) <(git show --format=fuller a49f225)<br>
>> --- /dev/fd/63  2013-11-02 13:40:30.964494674 -0600<br>
>> +++ /dev/fd/62  2013-11-02 13:40:30.964494674 -0600<br>
>> @@ -1,8 +1,8 @@<br>
>> -commit 53deab39b05c95f82ded0f9b5ed98e1eca77ec57<br>
>> +commit a49f225f21e5e022456a96fbd0a3a873045b2c5d<br>
>>  Author:     Peter Brune <<a href="mailto:brune@mcs.anl.gov">brune@mcs.anl.gov</a>><br>
>>  AuthorDate: Wed Oct 9 14:54:51 2013 -0500<br>
>>  Commit:     Peter Brune <<a href="mailto:brune@mcs.anl.gov">brune@mcs.anl.gov</a>><br>
>> -CommitDate: Wed Oct 9 14:54:51 2013 -0500<br>
>> +CommitDate: Thu Oct 17 13:38:04 2013 -0500<br>
>><br>
>>      Added several additional MPI_Ops to PetscSFBasic.<br>
>><br>
>><br>
>> $ git branch -r --contains a49f225<br>
>>   origin/prbrune/mat-matcolor<br>
>>   origin/prbrune/sf-sfbasicops<br>
>><br>
>><br>
>> Peter's rebased commit is a49f225, but it does exactly the same thing as<br>
>> the original (53deab3) and the only other branch it has been used by is<br>
>> 'prbrune/mat-matcolor', so it should be okay to:<br>
>><br>
>> $ git checkout prbrune/sf-sfbasicops<br>
>> $ git reset --hard f3fe83a        # gets rid of rebased version and merge<br>
>><br>
>><br>
>> $ git checkout prbrune/mat-matcolor<br>
>> $ git reset --hard HEAD^          # moves back to 3142415<br>
>> $ git merge prbrune/sf-sfbasicops<br>
>><br>
>><br>
>> Test that the above works, then force-push those two branches.<br>
>><br>
>> $ git push origin +prbrune/sf-sfbasicops +prbrune/mat-matcolor<br>
><br>
> barry/saws is now in master. This brought in prbrune/mat-matcolor<br>
> snapshot prior to the mpiuni fix into master.<br>
><br>
> How do we proceed here? [to get mpiuni fixes into master aswell]?<br>
> [i.e Peter do the above cleanup and merge to master - or merge to master as is?]<br>
<br>
</div></div>I would say Peter should do the above steps.  Or if he gives his okay,<br>
you or I can do it.<br>
</blockquote></div><br></div>