Skip to content

Make ghmerge useful for backports#238

Open
esyr wants to merge 1 commit into
openssl:masterfrom
esyr:esyr/ghmerge-rebase-base-onto
Open

Make ghmerge useful for backports#238
esyr wants to merge 1 commit into
openssl:masterfrom
esyr:esyr/ghmerge-rebase-base-onto

Conversation

@esyr

@esyr esyr commented Mar 23, 2026

Copy link
Copy Markdown
Member

Provide explicit base commit when rebasing the PR on the target branch; also, use the remote ref to avoid potential stray local commits.

Provide explicit base commit when rebasing the PR on the target branch;
also, use the remote ref to avoid potential stray local commits.

Signed-off-by: Eugene Syromiatnikov <esyr@openssl.org>
@paulidale

paulidale commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

What's inadequate with picktobranch?

@esyr

esyr commented Mar 23, 2026

Copy link
Copy Markdown
Member Author

Hm?

@paulidale

Copy link
Copy Markdown
Contributor

Apology, it's pick-to-branch. It is designed to do backports. It's not the same as ghmerge but it's easy enough to use.

@esyr

esyr commented Mar 23, 2026

Copy link
Copy Markdown
Member Author

Will take a look at it, thanks!

@t8m t8m left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually this might be handy sometimes.

@t8m t8m requested review from levitte, nhorman and paulidale June 16, 2026 10:00

@nhorman nhorman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just the one question. I think pick-to-branch is probably sufficient for this work, but this might be a handy additional tool to have in our pocket.

Comment thread review-tools/ghmerge
WHO=$1
BRANCH=$2
REPO=$3
BASE=$4

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of having to manually specify the base commit here for the rebase, would it be worth auto-computing it by running $(git merge-base HEAD), where base-branch is retrieved from the json of the pr being merged?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huh, the base branch is retrieved from the github PR here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants