Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade SpiderMonkey to mozilla-central commit 250be4ca3c669db9a397456402f68249aa15d8d5 #459

Merged
merged 9 commits into from
Jan 9, 2025

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 2, 2024

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Dec 2, 2024
@Xmader Xmader closed this Dec 3, 2024
@Xmader Xmader reopened this Dec 3, 2024
…oved from the definition in new SpiderMonkey version
…od on `JobQueue` for the new SpiderMonkey version
test_bytes_instanceof: Python bytes instanceof JS Uint8Array assert False

test_keys_iterator: TypeError iterator is not iterable

If `explicit-resource-management` is enabled, the lib .so will be compiled differently to the header files when there's a `IF_EXPLICIT_RESOURCE_MANAGEMENT` macro used.
The `enum JSProtoKey` index is off by 1 (header `JSProto_Uint8Array` 27 will be interpreted as `JSProto_Int8Array` in lib as lib has an extra element)
On https://searchfox.org/mozilla-central/rev/ead020d/mozglue/baseprofiler/public/BaseProfilerUtils.h#22, it detects Windows platform by seeing  if the `XP_WIN` macro is defined, if not then assumes it's on POSIX and includes the `<unistd.h>` header to use `getpid()`.

The `mozilla/BaseProfilerUtils.h` header file is introduced to `js/Debug.h` in https://phabricator.services.mozilla.com/D221102 (https://bugzilla.mozilla.org/show_bug.cgi?id=1911021), but it would be compiled without `XP_WIN` in our building configuration
Copy link
Member

@Xmader Xmader left a comment

Choose a reason for hiding this comment

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

Finally we fixed all the building issues!

@Xmader Xmader merged commit 02528c5 into main Jan 9, 2025
40 checks passed
@Xmader Xmader deleted the chore/upgrade-spidermonkey-to-250be4c branch January 9, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant