Skip to content

IEP-1792: Align execution environments to java 21#1488

Open
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792
Open

IEP-1792: Align execution environments to java 21#1488
sigmaaa wants to merge 1 commit into
masterfrom
IEP-1792

Conversation

@sigmaaa

@sigmaaa sigmaaa commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Description

align execution environments to java 21

Fixes # (IEP-XXX)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • ESP-IDF Version:
  • OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • Documentation

    • Updated setup, build, and contribution docs to require Java 21+.
    • Refreshed Chinese documentation to match the newer Java requirement.
  • Chores

    • Raised the minimum Java runtime across project configuration and bundle manifests to Java 21.
    • Updated Eclipse launch and classpath settings to use Java 21 for development and packaging.

@sigmaaa
sigmaaa requested a review from kolipakakondal July 8, 2026 10:30
@sigmaaa sigmaaa self-assigned this Jul 8, 2026
@sigmaaa sigmaaa changed the title feat: align execution environments to java 21 IEP-1792: Align execution environments to java 21 Jul 8, 2026
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c6d71093-4b92-467d-8be9-a988f8d85bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 0cfae1d and c3a8ecf.

📒 Files selected for processing (25)
  • AGENTS.md
  • CONTRIBUTING.md
  • README.md
  • README_CN.md
  • bundles/com.espressif.idf.branding/Espressif-IDE.launch
  • bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.debug.gdbjtag.openocd/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.help/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.launch.serial.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.lsp/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.core/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.sdk.config.ui/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.serial.monitor/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.swt.custom/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector.serial/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.terminal.connector/META-INF/MANIFEST.MF
  • bundles/com.espressif.idf.wokwi/META-INF/MANIFEST.MF
  • common/com.espressif.idf.nl.translator/.classpath
  • common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
  • internal/com.espressif.idf.uploads/.classpath
  • releng/com.espressif.idf.product/META-INF/MANIFEST.MF
  • releng/com.espressif.idf.product/idf.product
  • releng/com.espressif.idf.target/.classpath

📝 Walkthrough

Walkthrough

This PR raises the minimum required Java version from 17 (and in some cases 11) to 21 across documentation, OSGi bundle manifests, the Eclipse launch configuration, and Eclipse .classpath/product files, without altering any code logic or public APIs.

Changes

Java 21 upgrade

Layer / File(s) Summary
Documentation prerequisite updates
AGENTS.md, CONTRIBUTING.md, README.md, README_CN.md
Prerequisite sections and bundle-creation guidance updated to require Java 21+ instead of Java 17+.
Launch configuration and branding manifest
bundles/com.espressif.idf.branding/Espressif-IDE.launch, bundles/com.espressif.idf.branding/META-INF/MANIFEST.MF
Launch VM arguments switched to -Dosgi.requiredJavaVersion=21 and the branding bundle's Bundle-RequiredExecutionEnvironment set to JavaSE-21.
Bundle manifest RequiredExecutionEnvironment updates
bundles/com.espressif.idf.core/..., bundles/com.espressif.idf.debug.gdbjtag.openocd/..., bundles/com.espressif.idf.help/..., bundles/com.espressif.idf.launch.serial.core/..., bundles/com.espressif.idf.launch.serial.ui/..., bundles/com.espressif.idf.lsp/..., bundles/com.espressif.idf.sdk.config.core/..., bundles/com.espressif.idf.sdk.config.ui/..., bundles/com.espressif.idf.serial.monitor/..., bundles/com.espressif.idf.swt.custom/..., bundles/com.espressif.idf.terminal.connector.serial/..., bundles/com.espressif.idf.terminal.connector/..., bundles/com.espressif.idf.wokwi/..., common/com.espressif.idf.nl.translator/META-INF/MANIFEST.MF
Each bundle's Bundle-RequiredExecutionEnvironment header changed from JavaSE-17 to JavaSE-21.
Classpath and product configuration updates
common/com.espressif.idf.nl.translator/.classpath, internal/com.espressif.idf.uploads/.classpath, releng/com.espressif.idf.product/META-INF/MANIFEST.MF, releng/com.espressif.idf.product/idf.product, releng/com.espressif.idf.target/.classpath
JRE_CONTAINER entries in .classpath files, the product manifest, and the Solaris VM launcher entry updated to reference JavaSE-21 (from JavaSE-17 or JavaSE-11).

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: AndriiFilippov

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: updating execution environments to Java 21.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch IEP-1792

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant