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

Move data and templates to userdata subdirs #134

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ghisvail
Copy link
Contributor

@ghisvail ghisvail commented Nov 6, 2020

No description provided.

@ghisvail
Copy link
Contributor Author

ghisvail commented Nov 6, 2020

Assuming #133 is merged, that means data would end up under ${XDG_DATA_HOME}/higan-ui/Systems or ~/.local/share/higan-ui/Systems and templates under ${XDG_DATA_HOME}/higan-ui/Templates or ~/.local/share/higan-ui/Templates.

Path were chosen to remain consistent with the sample path.bml and avoid polluting $HOME with yet another application directory, which is what the XDG Base Directory Specification is here to avoid.

@ghisvail
Copy link
Contributor Author

ghisvail commented Nov 6, 2020

This PR probably is wrong as I don't get the point of the templates directory in this setup. I guess templates should point to (prefix)/share/higan where the content of the System folder is installed, correct?

@ghisvail
Copy link
Contributor Author

ghisvail commented Nov 8, 2020

Ok, I have looked at the packaging for Arch and Debian and both handle these paths in different ways:

  • On Arch, templates are installed to /usr/share/higan/Templates and data is set to $HOME/.local/higan/Systems
  • On Debian, templates are installed to /usr/share/higan and data is set to $HOME/.local/higan

The question is whether the data directory is supposed to evolve with more content, or is it supposed to only host systems defined by the user and nothing more. For the latter, the Debian way is fine. The Arch way is more flexible if desired.

@ghisvail ghisvail mentioned this pull request Nov 8, 2020
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.

1 participant