Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
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

roms built with latest uxn aren't working #2

Open
la-ninpre opened this issue Sep 30, 2021 · 7 comments
Open

roms built with latest uxn aren't working #2

la-ninpre opened this issue Sep 30, 2021 · 7 comments

Comments

@la-ninpre
Copy link

la-ninpre commented Sep 30, 2021

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.

@aduros
Copy link
Owner

aduros commented Sep 30, 2021

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.

@neauoire
Copy link

yup, it's just a matter of pulling the latest changes, it should work right off :)

@la-ninpre
Copy link
Author

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…

@aduros
Copy link
Owner

aduros commented Oct 1, 2021

You'll need at least emcc from emscripten, and nodejs to build the JS side.

@la-ninpre
Copy link
Author

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…

@neauoire
Copy link

neauoire commented Oct 2, 2021

Ah damn, that's true, the Layer is just a number(0-1) now.
The WIDTH and HEIGHT defines are still there?
https://git.sr.ht/~rabbits/uxn/tree/main/item/src/uxnemu.c#L24

@la-ninpre
Copy link
Author

hmm, WIDTH' and HEIGHT are there, but they are defined in uxn.c which is not used in webuxn. there is only uxn-fast.c.

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

3 participants