issues on RetroPie x86 about yarman HOT 10 OPEN

Ooops! Problems with dashboard in RetroPie x86:

from yarman.

Hmm, regarding the first point, I have only tested it on a pi. Have to check if php is also working on a pi, then we can change it :) or it needs some distinction between a pi and x86.

Second point should be the same issue. Only tested on pi. I will add some exception handling to it.

from yarman.

Let me know if you need some log file to help the dashboard fixing.

Cheers.

from yarman.

if you want you could test with the sqlite branch and add a screenshot of the dashboard. I have changed a couple of panels on it.

from yarman.

Using the sqlite branch I always get this popup:

I tried to Close, Sync and Sync with that checkbox checked... No matter what I choose the popup closes and instantaneously comes back.

from yarman.

Interesting that the background is empty. That shouldn't be.
Regarding the "sync behavior" I believe your path differs from the default "/home/pi/RetroPie/roms"? Therefore I have changed the setup process a bit :)

from yarman.

Regaring the empty dashboard, it looks like the problem is within the integrated php webserver... it is fixed now.

from yarman.

Regarding the "sync behavior" I believe your path differs from the default "/home/pi/RetroPie/roms"?

yeah, If you wanna make this tool available to other platforms than raspi you shouldn't hardcode the pi user homedir. But if you wanna limit it to raspi, the scriptmodule should specify the platforms to NOT install in rp_module_flags (this forum topic can be useful).

I make my tests/development in a x86 environment because it's on my laptop and I'm always carrying it with me, but I think it's useful for other users too.

I think the scriptmodule in the RetroPie-Setup PR is at a very embryonic stage. I'll try to make something similar to what I made for RetroPie-Manager and submit to the same branch of your Pull Request there.

from yarman.

Currently nothing is hardcoded :) but there is no config option for that yet. Defaults are set for raspi here. I already changed that at the setup process. I will add an admin panel later on as well for configuring.

I appreciate your support with the scriptmodule :) if it is required I can also close my PR there. As soon as it is merged in the official RetroPie-Setup I will remove the scriptmodule's in my repository as well.

from yarman.

if it is required I can also close my PR there.

It's not exactly required, but it can be a good idea since it would be cleaner to @joolswills review the code later.

from yarman.