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

quickjs for quickjs #137

Merged
merged 25 commits into from
Sep 7, 2024
Merged

quickjs for quickjs #137

merged 25 commits into from
Sep 7, 2024

Conversation

justjake
Copy link
Owner

@justjake justjake commented Jan 2, 2024

  • Add a build that can run inside quickjs, so you can put quickjs inside your quickjs
  • fix a build system bug that made commonjs or esm variants include both types, thus being larger than they needed to be

@twlite
Copy link

twlite commented Jan 18, 2024

😄 I will definitely put my quickjs inside quickjs inside quickjs when this is live

@justjake justjake force-pushed the jake--quickjs-in-quickjs branch from 28949e3 to 094fe7a Compare September 2, 2024 18:37
@justjake
Copy link
Owner Author

justjake commented Sep 2, 2024

@twlite unfortunately it doesn't seem to nest recursively, node -> quickjs-emscripten -> quickjs-for-quickjs is all the layers you get :(

@justjake justjake enabled auto-merge (squash) September 7, 2024 19:58
@justjake justjake merged commit 235f6da into main Sep 7, 2024
1 check passed
@justjake justjake deleted the jake--quickjs-in-quickjs branch September 7, 2024 20:15
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