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

Fix base path on Linux #66

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

nvllsvm
Copy link

@nvllsvm nvllsvm commented Apr 28, 2024

When the path is /usr/bin/nuked-sc55, the absolute path must be used to determine the correct base path of /usr/share/nuked-sc55.

When the path is /usr/bin/nuked-sc55, the absolute path must
be used to determine the correct base path of /usr/share/nuked-sc55.
@Wohlstand
Copy link
Collaborator

Should be fine!
Going to test this soon.

@UnBeatWaterGH

This comment was marked as outdated.

@nvllsvm
Copy link
Author

nvllsvm commented May 3, 2024

Hm - it works fine for me, though I'm able to reproduce your error after removing read permissions from the *.bin files. Do you still get the error after adding read permissions to the rom files (sudo chmod +r *bin)?

$ pwd
/home/ar
$ ls -l /usr/bin/../share/nuked-sc55
total 4396
-rw-r--r-- 1 root root  794352 Apr 28 19:29 back.data
-rw-r--r-- 1 root root   32768 Dec 24  1996 rom1.bin
-rw-r--r-- 1 root root  524288 Dec 24  1996 rom2.bin
-rw-r--r-- 1 root root    4096 Dec 24  1996 rom_sm.bin
-rw-r--r-- 1 root root 2097152 Dec 24  1996 waverom1.bin
-rw-r--r-- 1 root root 1048576 Dec 24  1996 waverom2.bin
$ nuked-sc55
Base path is: /usr/bin/../share/nuked-sc55
ROM set autodetect: SC-55mk2
Audio device: Default device
Audio Requested: F=S16LSB, C=2, R=66207, B=128
Audio Actual: F=S16LSB, C=2, R=66207, B=128
Unknown write e400 4
Unknown write e403 1
Unknown write e406 0
Unknown write e407 0
Unknown write e406 40
Unknown write e403 1

@UnBeatWaterGH
Copy link
Contributor

Works perfectly fine on Debian

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

Successfully merging this pull request may close these issues.

3 participants