Skip to content

Improve pgo-build.sh (backport #351)#352

Merged
qinsoon merged 2 commits intojdk-11from
mergify/bp/jdk-11/pr-351
Mar 26, 2026
Merged

Improve pgo-build.sh (backport #351)#352
qinsoon merged 2 commits intojdk-11from
mergify/bp/jdk-11/pr-351

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Mar 26, 2026

  1. Reduce hard-coded path/version/etc, making it easier to maintain and use in both jdk11 and jdk21.
  2. Add a step to configure jdk, so the script now works on fresh cloned repos.
    This is an automatic backport of pull request Improve pgo-build.sh #351 done by Mergify.

1. Reduce hard-coded path/version/etc, making it easier to maintain and
use in both jdk11 and jdk21.
2. Add a step to configure jdk, so the script now works on fresh cloned
repos.

(cherry picked from commit 4037ffa)

# Conflicts:
#	.github/scripts/pgo-build.sh
@mergify
Copy link
Copy Markdown
Author

mergify Bot commented Mar 26, 2026

Cherry-pick of 4037ffa has failed:

On branch mergify/bp/jdk-11/pr-351
Your branch is up to date with 'origin/jdk-11'.

You are currently cherry-picking commit 4037ffa.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   .github/scripts/pgo-build.sh

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@qinsoon qinsoon requested a review from wks March 26, 2026 06:51
Copy link
Copy Markdown
Contributor

@wks wks left a comment

Choose a reason for hiding this comment

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

LGTM

@qinsoon qinsoon merged commit d6d725a into jdk-11 Mar 26, 2026
55 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants