PaperWM icon indicating copy to clipboard operation
PaperWM copied to clipboard

path name with colon causes problems with gsettings and GSETTINGS_SCHEMA_DIR

Open wbolster opened this issue 4 years ago • 5 comments

this extension uses a : (colon) character in its directory name in ~/.local/gnome-shell/extensions, namely paperwm@hedning:matrix.org, which causes problems when using GSETTINGS_SCHEMA_DIR=... and the gsettings command line tool. the : is interpreted as a path separator.

this means gsettings cannot be used to configure this extension, as suggested in the readme.

since this is only the extension (out of quite a few on my system) using a non-standard character, i suggest to adhere to the established practices and conventions for gnome extensions, and use an identifier and directory name that do not cause problems.

wbolster avatar Jul 01 '20 08:07 wbolster

Right, that's a good point, the directory won't work in at least XDG_DATA_DIR which is colon separated. Now I'm fairly certain GSETTINGS_SCHEMA_DIR isn't colon separated, it's only meant to be used by a single directory, as the suggested dconf-editor hack in the readme works.

As an aside you can set things without a schema too (at least with dconf).

Will have a think about changing the id, and how a smooth upgrade can be handled.

hedning avatar Jul 01 '20 20:07 hedning

glib changed, and it only works with older versions

wbolster avatar Jul 01 '20 21:07 wbolster

Right, never noticed as I've only really used dconf-editor together with GSETTINGS_SCHEMA_DIR, and looking at the code it does some manual handling were it treats it as a single directory. So that's actually a bug in dconf-editor :p

hedning avatar Jul 01 '20 22:07 hedning

this popped up again in https://github.com/paperwm/PaperWM/issues/376#issuecomment-815561175

the best would be to just ‘get with the program’ :wink: and use the same naming conventions as pretty much all other gnome extensions. without a colon, that is.

wbolster avatar Apr 08 '21 08:04 wbolster

Yeah will just have to bite bullet here, propably do it for the 40 release

hedning avatar Apr 08 '21 08:04 hedning

To avoid future (potential) issues with this - yeah, I think we should change the id/name to [email protected]?

Will create a PR for this.

jtaala avatar Mar 11 '23 07:03 jtaala

Closing as uuid has been changed to safer [email protected].

jtaala avatar Aug 02 '23 10:08 jtaala