BHSPitMonkey / ninjam-js

The Ninjam music collaboration client reimplemented using JavaScript.
47 stars 7 forks source link

npm install in ninjam-js directory crashes iSH #9

Open canandlamp opened 1 year ago

canandlamp commented 1 year ago

simple as that

updated to latest npm after clearing cache to alleviate segmentation fault

all other ipad apps closed to free resources and ipad restarted, with time for wifi to come online

cd to ninjam-js directory in non-shared iSH file area placed git clone to avoid file permissions issue

npm install crashes iSH partway through, reliably

had been hoping to simply place build contents onto a personal webserver account in a private area to run what seemed to be all browser related code since getting a .crx file for chrome running on ipad seems to possibly require some shifty maneuvers to utilize safari's only ipad available browser extension mechanism, and seems to necessitate desktop hardware, so far as websearch declares

canandlamp commented 1 year ago

can the raw .js source and resources just be placed online as such in a private directory to draw off ninjam-js as though a standalone webpage?

canandlamp commented 1 year ago

pieced together url for crx download from webstore, renamed zip, uploaded contents to web server, set .map mimetype in htaccess, unblocked popups, on chrome and safari, just get a plain white webpage with permanent "loading..." message centered within. is this doable with some adjustments?

BHSPitMonkey commented 1 year ago

The short answer is no, this can't be simply run from a browser because the application relies on direct access to TCP sockets. The crx was a "Chrome App", not a browser extension, but Chrome Apps are no longer a supported feature so Electron is likely still the best way to run this app.

Note that this project hasn't been updated in years, so the versions of Electron and other libraries used are very outdated and likely insecure. With some effort it could be brought up to a modern Electron version and perhaps even made to run in a similar Node wrapper for mobile OSes, but this isn't something I have the time to assist with unfortunately.

kagel commented 5 months ago

I made a fork and fixed this issue, welcome. https://github.com/kagel/ninjam-js