Time |
Nickname |
Message |
01:30
🔗
|
|
db48x has joined #jsmess |
01:30
🔗
|
db48x |
mame is being annoying |
16:59
🔗
|
|
azakai has quit IRC (Quit: Ex-Chat) |
20:35
🔗
|
db48x |
DFJustin: what do you make of the error we get from https://archive.org/details/arcade_beezer? |
20:40
🔗
|
bai |
what devices is the machine configured with? |
20:41
🔗
|
bai |
is it just a standard arcade machine or is it one of those ones wich add-on cards and such? |
20:41
🔗
|
bai |
with* |
20:45
🔗
|
bai |
no peripherals or extra_args...hmm |
21:54
🔗
|
db48x |
yea, nothing special |
21:55
🔗
|
db48x |
the other thought I had was that I had failed to compile something in |
21:55
🔗
|
db48x |
but surely that would have been a link error |
22:11
🔗
|
DFJustin |
well in emscripten that manifests as a link warning rather than a link error |
22:11
🔗
|
DFJustin |
so you'd have to look at the build output to be sure |
22:12
🔗
|
DFJustin |
it could be an honest to goodness crash though which is a pain to debug |
22:16
🔗
|
DFJustin |
yep linker issue: warning: unresolved symbol: MM5837 |
22:21
🔗
|
DFJustin |
reproduces with native mame: |
22:21
🔗
|
DFJustin |
../../../../mingw-gcc/bin/x64/Release/mame_beezer/libmame_beezer.a(beezer.o):beezer.cpp:(.rdata$.refptr.MM5837[.refptr.MM5837]+0x0): undefined reference to `MM5837' |
22:23
🔗
|
* |
db48x sighs |
22:24
🔗
|
db48x |
a linker _warning_?! |
22:27
🔗
|
* |
db48x grumbles |
22:28
🔗
|
DFJustin |
using SOURCES=src/mame/drivers/beezer.cpp,src/devices/sound/mm5837.cpp should get it going but that's a legitimate mame bug |
22:29
🔗
|
DFJustin |
I don't know offhand how to fix it for real, it includes the .h which is supposed to do it |
22:31
🔗
|
db48x |
yes, I suppose I'll add something to the json so that the build script can tack on an extra filename |
22:31
🔗
|
DFJustin |
well I suspect it's a 5 minute fix for certain mamedevs |
22:31
🔗
|
db48x |
we'll also report it to them, of course |
22:36
🔗
|
db48x |
ok, beezer is fixed |
23:05
🔗
|
db48x |
grr |
23:06
🔗
|
db48x |
now when I build beezer, I don't have to specify mm5837.cpp |
23:55
🔗
|
DFJustin |
if you don't do REGENIE=1 changes to SOURCES don't take effect |