search

TomHarte / CLK

A latency-hating emulator of: the Acorn Electron and Archimedes, Amstrad CPC, Apple II/II+/IIe and early Macintosh, Atari 2600 and ST, ColecoVision, Enterprise 64/128, Commodore Vic-20 and Amiga, MSX 1/2, Oric 1/Atmos, early PC compatibles, Sega Master System, Sinclair ZX80/81 and ZX Spectrum.
MIT License
930 stars 52 forks source link

Building SDL version in Linux #943

Open LukeZBaker opened 3 years ago

LukeZBaker commented 3 years ago

I followed the instructions set out in BUILD.txt. The SDL version compiles fine on Ubuntu.

But even though I have all the ROMs in /usr/share/CLK/ (I also tried /usr/local/share/CLK/), the emulator complains that it can't find the ROM files. I also tried using the rompath flag pointing to where the ROM files are stored under the snap. Strangely that didn't work either, even though the Qt version works with them (with your help re colour.rom)

TomHarte commented 3 years ago

That it didn't succeed in searching /usr/share/CLK or /usr/local/share/CLK suggests something deeper afoot, and I think that as part of this I need to provide more specific feedback — i.e. the exact full paths that the emulator tried — but to ask an aside question: was it sufficiently clear that the command-line is --rompath=/whatever i.e. with the = sign in there?

TomHarte commented 3 years ago

The change just landed will print out the exact paths the emulator tried to use to look for any files it fails to find. That should at least help to determine whether the problem is in path conjugation, reading the command line, or elsewhere.

Please let me know if you get a chance to try it and, if so, whether it reveals anything.

iss000 commented 3 years ago

I'll reuse this open issue because it's not worth to open another one. After update to Linux Fedora 34 and g++ (GCC) 11.1.1 20210531 SDL compilation fails with reporting 'sizet' as undefined in some files. Attached patch fixes the issue by adding '#include _' where needed but I know you @TomHarte will solve it in better way. Huge thanks for your incredible work ;). clk-diff.patch.zip

TomHarte commented 3 years ago

Cool, thanks! I'll get it fixed.