8380993: [REDO] Incorrect Interpretation of POSIX TZ Environment Variable on AIX#607
8380993: [REDO] Incorrect Interpretation of POSIX TZ Environment Variable on AIX#607sruthyjay wants to merge 1 commit into
Conversation
Signed-off-by: Sruthy Jayan <srutjay1@in.ibm.com>
|
👋 Welcome back sruthyjay! A progress list of the required criteria for merging this PR into |
|
@sruthyjay This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be: You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 23 new commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details. As you do not have Committer status in this project an existing Committer must agree to sponsor your change. ➡️ To flag this PR as ready for integration with the above commit message, type |
|
This backport pull request has now been updated with issue from the original commit. |
|
|
|
/approval |
|
@sruthyjay usage: |
|
/approval JDK- 8380993 clean Backport request to restore correct AIX timezone mapping for modern POSIX TZ strings, preventing incorrect GMT fallback while maintaining backward compatibility with low-risk, AIX-only changes and regression test coverage. |
|
@sruthyjay usage: |
|
/approval JDK-8380993 request clean Backport to restore correct AIX timezone mapping for modern POSIX TZ strings, preventing incorrect GMT fallback while maintaining backward compatibility with low-risk, AIX-only changes and regression test coverage. |
|
@sruthyjay |
|
Hi @sruthyjay |
|
@sruthyjay Please enable GHA for your fork, thanks. See https://github.com/openjdk/jdk25u-dev/pull/607/checks?check_run_id=80972440379 |
|
/integrate |
|
@sruthyjay |
|
@sruthyjay Do you have a GHA run of this before I sponsor? |
|
@jerboaa I have triggered one now : https://github.com/sruthyjay/jdk25u-dev/actions/runs/28098615487 Thank you ! |
This one doesn't seem to be a run from your branch. |
Oops ! sorry .. https://github.com/sruthyjay/jdk25u-dev/actions/runs/28099385168 have triggered it for my branch .. I will cancel the master branch run. |
Thanks. This one shows up on this PR now. The windows build failure is known and should go away with a merged master run. I've triggered one here: https://github.com/jerboaa/jdk25u-dev/actions/runs/28099213050 |
|
GHA is clean. /sponsor |
|
Going to push as commit 567ecce.
Your commit was automatically rebased without conflicts. |
|
@jerboaa @sruthyjay Pushed as commit 567ecce. 💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored. |
This is a Backport PR : openjdk/jdk#31270
Progress
Issue
Reviewing
Using
gitCheckout this PR locally:
$ git fetch https://git.openjdk.org/jdk25u-dev.git pull/607/head:pull/607$ git checkout pull/607Update a local copy of the PR:
$ git checkout pull/607$ git pull https://git.openjdk.org/jdk25u-dev.git pull/607/headUsing Skara CLI tools
Checkout this PR locally:
$ git pr checkout 607View PR using the GUI difftool:
$ git pr show -t 607Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk25u-dev/pull/607.diff
Using Webrev
Link to Webrev Comment