Skip to content

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

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

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

Conversation

@mergify

@mergify mergify Bot commented Mar 26, 2026

Copy link
Copy Markdown
  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

mergify Bot commented Mar 26, 2026

Copy link
Copy Markdown
Author

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

@wks wks 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.

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