|Anonymous | Login | Signup for a new account||2018-08-14 06:42 UTC|
|My View | View Issues | Change Log | Roadmap | Doomseeker Issue Support Ranking | Rules | My Account|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0003243||Doomseeker||[All Projects] Bug||public||2017-09-01 16:17||2017-12-15 15:33|
|Platform||x86_64||OS||Debian GNU/Linux||OS Version||buster/sid|
|Target Version||Fixed in Version|
|Summary||0003243: Dubious default configuration path for Zandronum testing releases|
|Description||The default configuration sets TestingPath for Zandronum to $HOME/Documents/doomseeker/plugins/zandronum. Typically the downloaded testing releases include an executable binary and shared data (Skulltag PK3 files), which is a bit dubious to fit in the $XDG_DOCUMENTS_DIR directory.|
I expected $XDG_DOCUMENTS_DIR to be reserved for paper documents, literature at best.
|Steps To Reproduce|
|Additional Information||https://www.freedesktop.org/wiki/Software/xdg-user-dirs/ [^]|
I couldn't find an official specification related to XDG user directories. Needs support from issue 0003233 (XDG Base Directory Specification).
|Will QStandardPaths::DataLocation be a good place to store the builds, then?|
The documentation link you posted says QStandardPaths::DataLocation is deprecated and returns QStandardPaths::AppLocalDataLocation.
I know the torbrowser-launcher package in Debian GNU/Linux uses it ($HOME/.local/share/torbrowser) also for binaries, but that package is "not part of Debian" because of contrib section. It's acceptable, but not maybe the best choice. .local/share is meant to be for architecture independent files, after all.
Seeing how QStandardPaths::ApplicationsLocation is not always writable and it's used differently on Windows, it's also probably not a good choice.
I think QStandardPaths::AppLocalDataLocation ($HOME/.local/share/doomseeker) is still a more sensible choice, but that's arguable. Someone else may have a better idea.
|QStandardPaths::DownloadLocation or its subfolder may also be a sensible choice, excluding the generated shell script which should go to $HOME/.local/bin in my opinion. (Note, $HOME/.local/bin is not part of default $PATH.)|
|Eh, excluding the Qt options: I've personally kept my testing path to "$HOME/.local/games/zandronum-testing/", despite some shortcomings.|
Since this commit we now abstract the OS "local user data" location with Doomseeker's DataPaths::dataDirectory.
We can use dataDirectory to store plugin managed data by following a path pattern like this:
so, for testing releases it can be:
Example paths we would get:
Does this look right?
|That should be fine (for GNU/Linux at least).|
One caveat: Zandronum testing releases really shouldn't dump its arch-dependent binary and libs (x86 or x86_64) under one arch-independent .local/share directory, but that's out of scope for this ticket. This is partly an upstream issue of packaging too.
e.g. Python 2.7 uses $HOME/.local/lib/python2.7 (but has byte-compiled libs). nvim uses $HOME/.local/share/site/plugin (arch-independent scripts).
|I'll go with ~/.local/share, because it's better than Documents and QStandardPaths doesn't offer anything even better. No more #ifdefs, not for something that the user can change from the config box|
Zalewa: Yes, $HOME/.local/share is fine for now I think. The caveat I mentioned isn't "fixable" with the config option, and it's out of scope for this ticket.
For me, this ticket is resolved... for now, at least.
edited on: 2017-12-15 15:33
Note that this changes the behavior for --portable mode and testing binaries will now be installed to <progdir>/.static/plugins/zandronum.
|Only registered users can voice their support. Click here to register, or here to log in.|
|Supporters:||No one explicitly supports this issue yet.|
|Opponents:||No one explicitly opposes this issue yet.|
|2017-09-01 16:17||WubTheCaptain||New Issue|
|2017-09-01 16:31||Zalewa||Relationship added||child of 0003246|
|2017-09-09 19:00||Zalewa||Note Added: 0018251|
|2017-09-10 13:16||WubTheCaptain||Note Added: 0018256|
|2017-09-10 13:22||WubTheCaptain||Note Added: 0018257|
|2017-09-10 13:35||WubTheCaptain||Note Added: 0018259|
|2017-09-27 22:07||WubTheCaptain||Relationship added||child of 0003279|
|2017-09-27 22:07||WubTheCaptain||Relationship deleted||child of 0003246|
|2017-10-06 23:23||WubTheCaptain||Assigned To||=> WubTheCaptain|
|2017-10-06 23:23||WubTheCaptain||Status||new => acknowledged|
|2017-10-06 23:23||WubTheCaptain||Assigned To||WubTheCaptain =>|
|2017-12-11 15:56||Zalewa||Note Added: 0018951|
|2017-12-11 15:56||Zalewa||Assigned To||=> Zalewa|
|2017-12-11 15:56||Zalewa||Status||acknowledged => feedback|
|2017-12-12 11:42||WubTheCaptain||Note Added: 0018955|
|2017-12-12 11:42||WubTheCaptain||Status||feedback => assigned|
|2017-12-12 11:50||WubTheCaptain||Status||assigned => needs review|
|2017-12-12 12:03||WubTheCaptain||Note Added: 0018956|
|2017-12-12 16:06||Zalewa||Note Added: 0018958|
|2017-12-12 16:06||Zalewa||Status||needs review => assigned|
|2017-12-13 00:37||WubTheCaptain||Note Added: 0018960|
|2017-12-15 15:32||Zalewa||Note Added: 0018965|
|2017-12-15 15:32||Zalewa||Status||assigned => needs testing|
|2017-12-15 15:33||Zalewa||Note Edited: 0018965||View Revisions|
Questions or other issues? Contact Us.
|Copyright © 2000 - 2018 MantisBT Team|