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

[Next] Update should update not reinstall #26

Open
c-hri-s opened this issue Feb 27, 2018 · 5 comments
Open

[Next] Update should update not reinstall #26

c-hri-s opened this issue Feb 27, 2018 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@c-hri-s
Copy link

c-hri-s commented Feb 27, 2018

Hi,

Your update script takes a fairly brutal approach to upgrading, overwriting /boot/config.txt and so resetting overscan settings, and overwriting ES configuration settings so losing favourites, last played, skin, etc.

Is there a way to have it upgrade but leave modified files so user configuration/changes aren't lost?

Thanks

@kiteretro kiteretro changed the title Update should update not reinstall [Bug] Update should update not reinstall Feb 27, 2018
@kiteretro kiteretro self-assigned this Feb 27, 2018
@kiteretro kiteretro added the bug Something isn't working label Feb 27, 2018
@kiteretro
Copy link
Owner

Agreed.. so the following should probably happen:

I will probably add a flag or some file that tells the script "this is an update" because it is currently designed that install.sh is called and it has very basic settings applied..

For the time being you can do a git pull in the directory, and then edit those lines above to remove the changes you don't want and then run ./install.sh YES .. it is only the update.sh script that does the pull and then auto-install which you want to avoid for now..

@kiteretro kiteretro changed the title [Bug] Update should update not reinstall [Next] Update should update not reinstall Feb 27, 2018
@c-hri-s
Copy link
Author

c-hri-s commented Feb 27, 2018

Sounds good. Thanks for the workaround in the meantime.

@c-hri-s
Copy link
Author

c-hri-s commented Feb 27, 2018

You might like to consider rxbrad/gbz35 as the default theme instead of pixel. Might be a case of personal preference, but I find it much clearer on a small screen.

@kiteretro
Copy link
Owner

Thanks, I can pre-install it as part of the script and I'll try it out.. i've always used Pixel personally, but willing to give a go :) https://github.com/rxbrad/es-theme-gbz35

@jarettmillard
Copy link
Contributor

There's also Pixel TFT for small screens: https://github.com/djbriane/es-theme-pixel-tft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants