Skip to content

[25517] Change JavaScript engine to Mozilla rhino (backport #203) - #208

Merged
MiguelCompany merged 2 commits into
4.0.xfrom
hotfix/js-engine/4.0.x
Jul 22, 2026
Merged

[25517] Change JavaScript engine to Mozilla rhino (backport #203)#208
MiguelCompany merged 2 commits into
4.0.xfrom
hotfix/js-engine/4.0.x

Conversation

@MiguelCompany

@MiguelCompany MiguelCompany commented Jul 22, 2026

Copy link
Copy Markdown
Member

Description

This is a manual backport of #203 that gets the script engine with the explicit name rhino.

Contributor Checklist

  • Commit messages follow the project guidelines.
  • N/A: Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
  • N/A: Any new/modified methods have been properly documented.
  • Changes are backport compatible: they do NOT break ABI nor change library core behavior.
  • Changes are API compatible.
  • N/A: Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
…ild.gradle.

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany MiguelCompany added this to the v4.0.6 milestone Jul 22, 2026
@MiguelCompany
MiguelCompany marked this pull request as ready for review July 22, 2026 07:56

@ZakariaTalbi ZakariaTalbi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@MiguelCompany
MiguelCompany merged commit 696cfb3 into 4.0.x Jul 22, 2026
3 checks passed
@MiguelCompany
MiguelCompany deleted the hotfix/js-engine/4.0.x branch July 22, 2026 08:00
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.

2 participants