This is an automated email from the ASF dual-hosted git repository.
chaokunyang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/fory.git
The following commit(s) were added to refs/heads/main by this push:
new 8982e51a9 fix(java): fix maven deploy (#2452)
8982e51a9 is described below
commit 8982e51a9ac6c8dc9a0706c3c09a1c208c2a013a
Author: Shawn Yang <[email protected]>
AuthorDate: Mon Aug 11 13:44:52 2025 +0800
fix(java): fix maven deploy (#2452)
<!--
**Thanks for contributing to Fory.**
**If this is your first time opening a PR on fory, you can refer to
[CONTRIBUTING.md](https://github.com/apache/fory/blob/main/CONTRIBUTING.md).**
Contribution Checklist
- The **Apache Fory** community has requirements on the naming of pr
titles. You can also find instructions in
[CONTRIBUTING.md](https://github.com/apache/fory/blob/main/CONTRIBUTING.md).
- Fory has a strong focus on performance. If the PR you submit will have
an impact on performance, please benchmark it first and provide the
benchmark result here.
-->
## What does this PR do?
<!-- Describe the purpose of this PR. -->
## Related issues
<!--
Is there any related issue? Please attach here.
- #xxxx0
- #xxxx1
- #xxxx2
-->
## Does this PR introduce any user-facing change?
<!--
If any user-facing interface changes, please [open an
issue](https://github.com/apache/fory/issues/new/choose) describing the
need to do so and update the document if necessary.
-->
- [ ] Does this PR introduce any public API change?
- [ ] Does this PR introduce any binary protocol compatibility change?
## Benchmark
<!--
When the PR has an impact on performance (if you don't know whether the
PR will have an impact on performance, you can submit the PR first, and
if it will have impact on performance, the code reviewer will explain
it), be sure to attach a benchmark data here.
-->
---
.github/workflows/release-java-snapshot.yaml | 4 ++--
ci/tasks/java.py | 6 +-----
2 files changed, 3 insertions(+), 7 deletions(-)
diff --git a/.github/workflows/release-java-snapshot.yaml
b/.github/workflows/release-java-snapshot.yaml
index 5e63abad3..1292ccc1f 100644
--- a/.github/workflows/release-java-snapshot.yaml
+++ b/.github/workflows/release-java-snapshot.yaml
@@ -32,7 +32,7 @@ jobs:
- name: Set up Maven Central Repository
uses: actions/setup-java@v4
with:
- java-version: "8"
+ java-version: "11"
distribution: "adopt"
architecture: x64
cache: maven
@@ -40,7 +40,7 @@ jobs:
server-username: NEXUS_USERNAME
server-password: NEXUS_PASSWORD
- name: Publish Fory Java Snapshot
- run: python ./ci/run_ci.py java --version 8 --release
+ run: python ./ci/run_ci.py java --version 11 --release
env:
NEXUS_USERNAME: ${{ secrets.NEXUS_USER }}
NEXUS_PASSWORD: ${{ secrets.NEXUS_PW }}
diff --git a/ci/tasks/java.py b/ci/tasks/java.py
index 9dce623e7..38d646244 100644
--- a/ci/tasks/java.py
+++ b/ci/tasks/java.py
@@ -286,10 +286,6 @@ def run_graalvm_test():
def run_release():
"""Release to Maven Central."""
logging.info(f"Starting release to Maven Central with Java")
- install_jdks()
- java_home = os.path.join(common.PROJECT_ROOT_DIR, JDKS["8"])
- os.environ["JAVA_HOME"] = java_home
- os.environ["PATH"] = f"{java_home}/bin:{os.environ.get('PATH', '')}"
common.cd_project_subdir("java")
# Clean and install without tests first
@@ -310,7 +306,7 @@ def run(version=None, release=False, install_jdks=False,
install_fory=False):
if install_fory:
globals()["install_fory"]()
if release:
- logging.info("Release mode enabled - will release to Maven Central")
+ logging.info("Release mode enabled - will release to Maven Repository")
run_release()
elif version == "8":
run_java8()
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
