-
Notifications
You must be signed in to change notification settings - Fork 6
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
roms built with latest uxn aren't working #2
Comments
I think there have been upstream breaking changes to uxn. It's hopefully just a matter of pulling in the latest C sources (uxn-fast.c, ppu.c, etc) from upstream though. If you get it working, can you please open a PR? cc @neauoire who probably knows more about it. |
yup, it's just a matter of pulling the latest changes, it should work right off :) |
i haven't get it working yet. i'll try, though. and if it'll be successfull, i'll make a pull request. oh, and also, kinda aside question: on which platform and using what tools webuxn is supposed to be built? i'm asking because the build fails for me in archlinux… |
You'll need at least |
hmm, maybe i'm doing something wrong, but when i put new uxn-fast.c, apu.{h,c}, ppu.{h,c} and uxn.h it doesn't work at all. it seems that a lot of functions now have different names and even different prototypes. like, really, there are a lot of changes and that needs a bit of research, because, for example, there are no Layer struct in new uxn, there are no WIDTH and HEIGHT defines and so on… |
Ah damn, that's true, the Layer is just a number( |
hmm, |
i tried to run webuxn locally, then i loaded piano.rom which i just compiled with latest version of uxn and it is just a black screen.
for comparison, if i take piano.rom from this repo, it runs fine.
i am not at all an expert in wasm and stuff, but maybe there was some incompatible change in upstream uxn?
both roms run fine with uxnemu though...UPD: tested old roms with current uxnemu, and they aren't working.
The text was updated successfully, but these errors were encountered: