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.

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

BASE_PATH should be INI_PATH in the .ini? #109

Open
scottglazer opened this issue Jul 29, 2024 · 0 comments
Open

BASE_PATH should be INI_PATH in the .ini? #109

scottglazer opened this issue Jul 29, 2024 · 0 comments

Comments

@scottglazer
Copy link

I followed the instructions given on the main page, but step 5 kept failing:

change the .ini file to the base path (BASE_PATH=/media/fat) where your vhds are located, /media/fat, /media/usb0, /media/fat/games/cifs, etc

Bizarrely, the base path reported in the error when the script failed would contain the path in duplicate, IE:

/media/fat/games/AO486
appeared as
/media/fat/games/AO486/media/fat/games/AO486

The .ini did have a BASE_PATH value I was editing, but when the script ran it could never find the .vhd file. I had a look at the shell script, and it looks like you're looking for INI_PATH rather than BASE_PATH:

INI_PATH="${ORIGINAL_SCRIPT_PATH%.*}.ini"

...editing the .ini to use INI_PATH fixed the script and allowed it to work. I'd think either updating the instructions to indicate the new variable name or changing the variable name itself might be the way to go? Unless of course I resolved the issue for a reason outside what my limited knowledge of scripting entails - maybe editing to INI_PATH just caused the script to fall back on a default value or something, since it no longer had a BASE_PATH entry and I got lucky.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant