Skip to content

do not fix the JDK version in the swig rules#7352

Merged
waruqi merged 2 commits intoxmake-io:devfrom
binLep:dev
Feb 27, 2026
Merged

do not fix the JDK version in the swig rules#7352
waruqi merged 2 commits intoxmake-io:devfrom
binLep:dev

Conversation

@binLep
Copy link
Contributor

@binLep binLep commented Feb 26, 2026

我先把这个 release 版本的问题交了

这里固定了 jdk 版本为 17,用 jdk 1.8 的话会报错,删了就好了

@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @binLep, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

此拉取请求解决了 SWIG 规则中与 Java 开发工具包 (JDK) 版本兼容性相关的构建问题。它从 javac 编译命令中移除了硬编码的 JDK 17 发布标志,从而允许使用较旧的 JDK 版本(如 1.8)进行构建。这提高了灵活性,并防止了非 JDK 17 用户遇到编译错误。

Highlights

  • JDK 版本兼容性: 移除了 SWIG 规则中 javac 命令的固定 JDK 版本限制。
  • 构建修复: 解决了使用 JDK 1.8 时因硬编码 JDK 17 版本而导致的编译失败问题。

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • xmake/rules/swig/build_module_file.lua
    • javac 命令中移除了 --release 17 标志。
Activity
  • 没有提供任何活动信息。
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

这个 PR 移除了在 swig 规则中硬编码的 JDK 版本 17,解决了在使用旧版本 JDK (如 1.8) 时导致的编译错误。这是一个正确的修复。我还建议了一个改进,使其更加灵活,允许用户在需要时通过 xmake.lua 配置来指定 Java 的 release 版本。

@waruqi waruqi added this to the v3.0.8 milestone Feb 26, 2026
@waruqi
Copy link
Member

waruqi commented Feb 26, 2026

可以按照 genmini 说的,加个 java version 的指定参数。。有时候一些 java 模块可能需要限定 java 版本。

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


You can follow what genmini said and add a java version specified parameter. . Sometimes some java modules may need to limit the java version.

@waruqi waruqi merged commit 71e10b7 into xmake-io:dev Feb 27, 2026
37 checks passed
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.

3 participants