Skip to content

Switch JS Engine from SpiderMonkey to QuickJS#228

Merged
manugarg merged 6 commits intomainfrom
migrate_quickjs
Feb 8, 2026
Merged

Switch JS Engine from SpiderMonkey to QuickJS#228
manugarg merged 6 commits intomainfrom
migrate_quickjs

Conversation

@manugarg
Copy link
Copy Markdown
Owner

@manugarg manugarg commented Feb 7, 2026

No description provided.

manugarg and others added 4 commits February 7, 2026 17:59
The previous approach copied pre-compiled libquickjs.a and pacparser.o
from the host (ubuntu-latest) into the manylinux container. This caused
auditwheel to fail because the objects were built against a newer glibc
than the manylinux ABI allows.

Now we copy source files and use CIBW_BEFORE_BUILD to compile
libquickjs.a and pacparser.o inside the manylinux container with the
appropriate toolchain.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The host pymod build creates artifacts in build/ that cibuildwheel
picks up, causing auditwheel to fail. Clean them first.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 8, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@manugarg manugarg changed the title Migrate pacparser to quickjs Migrate pacparser from spidermonkey to quickjs Feb 8, 2026
@manugarg manugarg changed the title Migrate pacparser from spidermonkey to quickjs Switch JS Engine from SpiderMonkey to QuickJS Feb 8, 2026
@manugarg manugarg merged commit ff64256 into main Feb 8, 2026
29 of 30 checks passed
@manugarg manugarg deleted the migrate_quickjs branch February 8, 2026 10: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.

1 participant