Skip to content

script: Bump to mozjs with safe APIs - #46461

Merged
Gae24 merged 2 commits into
servo:mainfrom
Gae24:mozjs-bump
Jul 14, 2026
Merged

script: Bump to mozjs with safe APIs#46461
Gae24 merged 2 commits into
servo:mainfrom
Gae24:mozjs-bump

Conversation

@Gae24

@Gae24 Gae24 commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Switch to safe variants of describe_scripted_caller and error_info_from_exception_stack.

Testing: It compiles

@Gae24
Gae24 requested a review from gterzian as a code owner July 12, 2026 21:37
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 12, 2026
@Gae24 Gae24 changed the title script: Bump mozjs with safe APIs script: Bump to mozjs with safe APIs Jul 12, 2026
@Narfinger

Copy link
Copy Markdown
Contributor

Looks good (when mozjs is merged).

sagudev pushed a commit to sagudev/mozjs that referenced this pull request Jul 14, 2026
To fully remove unsafe bindings we need to introduce safe variants of
some existing mozjs APIs, starting with `describe_scripted_caller` and
`error_info_from_exception_stack`.
Deprecate `wrappers` module too.

Testing: It compiles
Servo PR: servo/servo#46461

---------

Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Gae24 added 2 commits July 14, 2026 20:25
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
Signed-off-by: Gae24 <96017547+Gae24@users.noreply.github.com>
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jul 14, 2026
@Gae24
Gae24 added this pull request to the merge queue Jul 14, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 14, 2026
Merged via the queue into servo:main with commit 97cc7f7 Jul 14, 2026
31 checks passed
@Gae24
Gae24 deleted the mozjs-bump branch July 14, 2026 23:57
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 14, 2026
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.

4 participants