search

1j01 / 98

💿 Web-based Windows 98 desktop recreation █████▓█▓▓▒▓▒▒░▒░░░🗕︎🗗︎🗙︎
https://98.js.org
1.15k stars 183 forks source link

"The filesystem is not working" #77

Open bhsuarez opened 2 years ago

bhsuarez commented 2 years ago

I'm getting an npm error when I do npm start

make_http_index | json > filesystem-index.json sh: make_http_index: command not found

Is this an NPM error?

bhsuarez commented 2 years ago

I figured it out. Here you state that browserfs@2.0.0 is currently assumed to be installed globally but in the dependencies list, you put browserfs@1.4.3. I guess that wasn't a good assumption cuz I've never done anything with browserfs before and once I changed the dependency value to the 2.0.0 version, the error didn't show up on npm start

Is there a reason for the discrepancy? If not I will do a fork.

1j01 commented 2 months ago

I had patched browserfs@1.4.3, which is why I didn't want to upgrade, but I still needed the make_http_index CLI only available in browserfs@2.0.0. That's how I came to depend on two separate versions of the same package.

I've now documented this on the readme, and I'm working on migrating to ZenFS, a fork of BrowserFS.

I guess I could have also used package aliases to avoid needing a global install, since I only needed the command from one of the versions, and the other version didn't have the command. That's assuming npm 6.9.0 was out at the time.