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

Allow config path configuration on linux #963

Closed
Moredread opened this issue Jun 9, 2018 · 13 comments
Closed

Allow config path configuration on linux #963

Moredread opened this issue Jun 9, 2018 · 13 comments

Comments

@Moredread
Copy link

Version

1.40.0-beta

Operating system type + version

  • system: "x86_64-linux"
  • host os: Linux 4.16.14, NixOS, 18.03.132618.0f73fef53a9 (Impala)
  • multi-user?: yes
  • sandbox: yes
  • version: nix-env (Nix) 2.1pre6148_a4aac7f
  • channels(root): "nixos-18.03.132618.0f73fef53a9, nixos-unstable-18.09pre140958.696c6bed4e8"
  • channels(addy): "nixos-18.03"
  • nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Behavior

On linux the user configuration currently is saved in ~/.Slic3rPE.

It would be nice though if it would follow the XDG Base Directory Specification, esp. the convention for config directories.

This would reduce clutter in the users home directory, and also would allow the user to choose where she saves her config files (via setting the $XDG_CONFIG_HOME).

This is a feature request.

@Moredread
Copy link
Author

See slic3r/Slic3r#832 in the upstream repo, with a possible solution.

@vojtechkral
Copy link
Contributor

It would be nice if we could follow the XDG recommendation and it would be fairly easy to implement, however, there's the issue of continuity / backwards compatibility. Users already have their configs in ~/.Slic3rPE and expect them to be there... I'm not sure how we'd go about that.

@bubnikv
Copy link
Collaborator

bubnikv commented Jun 11, 2018 via email

@iicurtis
Copy link

This is now relevant again. Since the rename, we already break backward compatibility.

@bubnikv
Copy link
Collaborator

bubnikv commented May 16, 2019

Are you compiling PrusaSlicer yourself?
Could you try the following?
https://docs.wxwidgets.org/trunk/classwx_standard_paths.html#af5eb9a009b9fd81b748e26d099bb4e89

@bubnikv
Copy link
Collaborator

bubnikv commented May 16, 2019

On the build server to build our binary packages we are using the wxWidgets 3.1.1. This version should support XDG automatically

wxString wxStandardPaths::GetUserDir(Dir userDir) const
{
// Note that we do not use the file layout here because there is no reason
// not to respect the XDG convention even if SetFileLayout(FileLayout_XDG)
// hadn't been called: we're not bound by any backwards compatibility
// considerations as there can't be any pre-existing config or data files
// in the home directory that wouldn't be found any longer after updating
// the version of wxWidgets used by the application.

https://docs.wxwidgets.org/trunk/classwx_standard_paths.html#a86bd7b0187fd299826060f06c7cff5ed

Does it work for you?

@iicurtis
Copy link

I am sorry, I don't quite understand what you want me to test. The current binary still creates a new directory ~/.PrusaSlicer. I am using wxgtk3 3.0.4 however.

@bubnikv
Copy link
Collaborator

bubnikv commented May 16, 2019

I want you to test the binary slicer (appimage) that we compiled using wxWidgets 3.1.1.

@iicurtis
Copy link

I have tested both the binary in the .tar.bz2 and the appimage. Both create a directory ~/.PrusaSlicer instead of using ~/.config.

@guestisp
Copy link

guestisp commented May 16, 2019

I can confirm.

$ ls -lad ~/.PrusaSlicer
drwxr-xr-x 10 x x 4096 mag 16 13:07 /home/x/.PrusaSlicer
$
$ ls -la ~/.config/ | grep -i prusa
$ 

@Schnilz
Copy link

Schnilz commented Oct 3, 2019

any news on this ?

@wackbyte
Copy link

Any more news on this?

@lukasmatena
Copy link
Collaborator

This was implemented for PrusaSlicer 2.3.0-alpha2, see #2911. XDG specification should now be respected, users will get an alert when they first run the new version so they know where their config went and where to move it.

Closing.

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

No branches or pull requests

8 participants