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

change save directory to ~/.local/share/pokete, and add migrate.sh to copy the data over #161

Merged
merged 7 commits into from Jun 20, 2022

Conversation

greatericontop
Copy link
Contributor

It's not a great idea to set it to ~/.cache, because this directory is supposed to be temporary and could be cleared every so often. People also sometimes clear it as it tends to get quite large over time.

… copy the data over

using ~/.cache is bad because it can be cleared and people delete them sometimes
Copy link
Owner

@lxgr-linux lxgr-linux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since it was requested here #137, I think it's a good idea to use the $XDG_DATA_HOME environment variable to determine the save location.

release.py Show resolved Hide resolved
release.py Outdated Show resolved Hide resolved
@lxgr-linux lxgr-linux requested a review from MaFeLP June 15, 2022 15:17
@lxgr-linux
Copy link
Owner

@daynetran

Copy link
Owner

@lxgr-linux lxgr-linux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also in that case all mentions of HOME in pokete.py should be removed, because the savepath now is a absolute path and not relative to ~ anymore. Also the addition of a .bat script to migrate the windows user may be helpful.

Co-authored-by: lxgr-linux <64218491+lxgr-linux@users.noreply.github.com>
@MaFeLP
Copy link
Collaborator

MaFeLP commented Jun 15, 2022

Thank you for your contribution.

I would just like to have it fully backwards compatible, which means for me no user intervention required.
I would therefore love to see a check, if data in the SAVEPATH exists, and if not, try to load it from the old save location. If there also is no data to be found, start a 'fresh' instance of pokete.

@lxgr-linux
Copy link
Owner

Ready for review @MaFeLP

@lxgr-linux lxgr-linux linked an issue Jun 19, 2022 that may be closed by this pull request
2 tasks
@lxgr-linux lxgr-linux merged commit bc92971 into lxgr-linux:master Jun 20, 2022
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

Successfully merging this pull request may close these issues.

Need to create the pokete directory in $HOME/.cache
3 participants