Re: rxe MW PR for rdma-core

[Bob Pearson <rpearsonhpe@gmail.com>]

On 6/25/21 9:39 AM, Jason Gunthorpe wrote:
> On Thu, Jun 24, 2021 at 11:59:14PM -0500, Bob Pearson wrote:
> 
>> It took a few hours but I finally managed to get rid of the merge
>> commits in the rxe MW PR. It's back out there at github. I lost the
>> name change Jason had made when I deleted and reforked my repo. I
>> remade the update kernel headers commit without the ??. It still
>> passes all the screening tests.
> 
> You should never have to delete and refork with git.
> 
> Force push fixes all mistakes.
> 
> The sequence to fix your situation is
> 
> 1) Starting at the bad branch merge to latest rdma-core
> 2) Create a new branch on latest rdma-core
> 3) Use 'git cherry-pick' on each non-merge commit from the
>    bad branch
> 4) Diff the bad/good branch to make sure nothing got missed in the
>    merges
> 5) Reset the bad branch to the new branch's commit ID.
> 6) Force push the fixed branch to github.
> 
> In future use 'git rebase' instead of 'git merge'
> 
> Jason
> 
I have a question about how the various trees are arranged.
I started by cloning the rdma-core tree from github to my local machine (i.e. origin master).
Then later I added my own github account and forked the rdma-core tree.
I have a link to is in my local .git/config (i.e. my-rdma-core).
Normally I do a git pull from origin which is shorthand for git fetch + git merge (I think??)
To rebase instead do you have to first do a fetch then a rebase or can you set the rebase flag in
.git/config?
I also do updates in the github web site from rdma-core to my copy there. I was having a hard time
getting the private github tree and the local tree to match (because of merge commits)

In the github docs that I read last evening they seem to recommend cloning the forked github tree
instead of the upstream tree on your local system. Is this correct?

Also in 5) above how do you do that?

thanks

bob