-
Notifications
You must be signed in to change notification settings - Fork 132
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
Disk images are pulled from internet when running locally #37
Comments
OK, the Line 29 in 13bad3f
removing that line fixes the issue. |
Apologies for not checking on open issues more regularly. Glad you were persistent and found a resolution. When I'm running locally, I'm usually doing it through VSCode, which I've configured to start Jekyll using the "bundle serve uncompiled" task in tasks.json, which includes _developer.yml and overrides _config.yml with, um, less exclusive excludes. I've updated Running PCjs Locally with that small detail. |
Thanks for the update! Unfortunately, simply adding This is how I start it:
And then I go to http://localhost:4000/configs/pcx86/machine/compaq/deskpro386/vga/4096kb/machine.xml. |
The "Running locally" guide mentions that when doing so local disk images are used, but I don't see it to be the case:
From what I managed to understand, the following code is responsible for this:
Unfortunately, the actual web page uses its "compiled" version, where this check is thrown away which forces it not to even consider trying fetching the disk locally:
Now, looking at https://github.com/jeffpar/pcjs/tree/master/machines/pcx86/releases, it seems there are many different versions of that
pcx86.js
file and it seems different images use different versions and some of them even reference "uncompiled" versions of it. How does it happen?How do I fix this?
Ultimately, I want my custom diskette image loaded. I added it to
/configs/pcx86/diskettes.json
and I see it appear in the drop down box, but when I actually try to load it, the request is sent todiskettes.pcjs.org
where it obviously does not exist.The text was updated successfully, but these errors were encountered: