Skip to content
This repository has been archived by the owner on Jan 9, 2019. It is now read-only.

issues on RetroPie x86 #33

Open
meleu opened this issue Mar 5, 2017 · 10 comments
Open

issues on RetroPie x86 #33

meleu opened this issue Mar 5, 2017 · 10 comments
Labels

Comments

@meleu
Copy link
Contributor

meleu commented Mar 5, 2017

I tried to install yarman using your scriptmodule in my RetroPie x86 and the the packages php5 and php5-sqlite don't exist in the official Ubuntu/Linux Mint repositories...

Then I changed the script to install php and php-sqlite and it seems that yarman is able to run...

@meleu
Copy link
Contributor Author

meleu commented Mar 5, 2017

Ooops! Problems with dashboard in RetroPie x86:
captura de tela de 2017-03-04 22-29-00

@daeks
Copy link
Member

daeks commented Mar 6, 2017

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.

@meleu
Copy link
Contributor Author

meleu commented Mar 6, 2017

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

Cheers.

@daeks daeks added the bug label Mar 6, 2017
@daeks
Copy link
Member

daeks commented Mar 6, 2017

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.

@meleu
Copy link
Contributor Author

meleu commented Mar 7, 2017

Using the sqlite branch I always get this popup:
captura de tela de 2017-03-06 23-24-28

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

@daeks
Copy link
Member

daeks commented Mar 7, 2017

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 :)

@daeks
Copy link
Member

daeks commented Mar 7, 2017

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

@meleu
Copy link
Contributor Author

meleu commented Mar 7, 2017

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.

@daeks
Copy link
Member

daeks commented Mar 7, 2017

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.

image

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.

@meleu
Copy link
Contributor Author

meleu commented Mar 7, 2017

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

2 participants