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