You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
(fix): slightly modify ammo so it won't try to require stuff
- this is what I was having trouble with when I first created this repo
- c.f. kripken/ammo.js#109 (comment) ,
webpack/webpack#7352 ,
and emscripten-core/emscripten#6542
- can't just upgrade to a new ammo as that might very well break
physijs, so just make the Node environment check false
- in this case I changed `var qa="object"===typeof process,` to
`var qa=false,` in the minified code
- and also remove thed `if(qa){`...`}` parts
- this was the part that had `require` statements, so now webpack
etc should be able to parse it without a problem
- there was a `require("fs")` and a `require("path")` in there
- alternatively, could build and replace stuff with webpack, but I'd
need to provide that anyway for auto-config
- downside is it might no longer work on Node, but not the target
audience so w/e
0 commit comments