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.

Sign up for GitHub

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

NPM and Typescript support #647

Draft
wants to merge 14 commits into
base: master
Choose a base branch
from
+663 −79
Draft

NPM and Typescript support #647

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
caaba6b
package.json and .npmrc for github package registry
ambientlight Feb 2, 2022
2b95862
base typescript definitions
ambientlight Feb 5, 2022
d7dafb6
expose bus
ambientlight Feb 10, 2022
f793b31
Merge branch 'copy:master' into feat/ts
ambientlight May 3, 2022
5e0a5ab
Merge branch 'copy:master' into feat/npm
ambientlight May 3, 2022
9e0a2c1
Merge branch 'feat/ts' into feat/npmts_upstream
ambientlight May 4, 2022
d65220d
include types in package.json
ambientlight May 4, 2022
a49e2b5
add bios and debug bundled to files
ambientlight May 4, 2022
7176ad2
remove github registry .npmrc, change links to @copy
ambientlight May 4, 2022
af37e1d
check for commonjs first in exports statements, commonjs exports for …
ambientlight May 14, 2022
2e7686c
v86 module in typings
ambientlight May 14, 2022
2615213
address review comments
ambientlight May 14, 2022
1c34108
change package name and version
ambientlight May 14, 2022
a237df8
distribute with libwabt and capstone-x86
ambientlight May 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"build": "npm run build:all_debug & npm run build:all",
"build:all_debug": "make all-debug",
"build:all": "make all",
"build:release": "make build/libv86.js build/v86.wasm build/v86-fallback.wasm",
"build:release": "make build/libv86.js build/v86.wasm build/v86-fallback.wasm build/capstone-x86.min.js build/libwabt.js",
Copy link
Owner

Choose a reason for hiding this comment

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

These are only for debugging, I don't think they should be included in a release build.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the bundlers will parse the debug.js requires by defaults and would try to bundle those - if we don’t distribute them the bundlers would complain unless we add explicit externs, my thought here was is to make bundling succeed on defaults, but we can also do something so that the compiled code does not contain require() on those two libraries.

Copy link
Owner

Choose a reason for hiding this comment

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

I see. If you add if(!DEBUG) return; to the beginning of the functions that call require that should cause closure compiler to remove those calls. Alternatively, I'd fine with removing the require call entirely. I haven't used those libraries from node in a long time anyway.

"test:prepare": "wget -nv -P images/ https://k.copy.sh/{linux.iso,linux3.iso,linux4.iso,buildroot-bzimage.bin,TinyCore-11.0.iso,oberon.img,msdos.img,openbsd-floppy.img,kolibri.img,windows101.img,os8.img,freedos722.img,mobius-fd-release5.img}",
"test": "make kvm-unit-test nasmtests nasmtests-force-jit expect-tests jitpagingtests qemutests qemutests-release rust-test tests"
},
Expand All @@ -28,6 +28,8 @@
"build/v86.wasm",
"build/v86-debug.wasm",
"build/v86-fallback.wasm",
"build/capstone-x86.min.js",
"build/libwabt.js",
"types/index.d.ts"
],
"types": "types/index.d.ts",
Expand Down