Yeah and let's slow our roll here. We might want to see if lxqt-globalkeys doesn't actually support XDG. It should. If so, this can go in default settings after all.

One thing you could do while we figure this out, though, is to clean up your diff as aforementioned.

So I've confirmed it is hardcoded and it seems to be a known problem but it's kind of a bugaboo. @tsimonq2 maybe we should just go the route of patching globalkeys now and later when we get the patch to respect XDG, we can move that patch to default-settings?

Regarding the patch, remove this:

Origin: <upstream|backport|vendor|other>, <URL, required except if Author is present>
Bug: <URL to the upstream bug report if any, implies patch has been forwarded, optional>
Bug-<Vendor>: <URL to the vendor bug report if any, optional>
Forwarded: <URL|no|not-needed, useless if you have a Bug field, optional>
Applied-Upstream: <version|URL|commit, identifies patches merged upstream, optional>
Reviewed-by: <name and email of a reviewer, optional>
---
This patch header follows DEP-3: http://dep.debian.net/deps/dep3/

Your date has some extra cruft, too, and should just be:

Last-Update: 2019-07-02

I'd recommend adding:

Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/lubuntu-default-settings/+bug/1802501

Finally, I think we need to make a decision about whether or not we *are* patching this again here or whether or not we should wait for full XDG support upstream. @tsimonq2 hello???

Looks like there's a potential upstream pull request for making globalkeys respect XDG, so this may end up going to rDEFAULTSETTINGS after all.

So..? We wait then?

We test to see if it works. If it does, then we will need to prioritize that first. I doubt we will get a new release, so we'll have to patch our version in Ubuntu. Once that's settled, then we deal with putting this in the right place. According to the Qt docs, barring a local config, QSettings (which is what the pull request uses) will look through XDG_CONFIG_DIRS. As our wiki implies, this should equal /etc/xdg/xdg-Lubuntu:/etc/xdg:/etc:/usr/share. rDEFAULTSETTINGS already has a /etc/xdg/xdg-Lubuntu/lxqt folder so that's where this would end up if this does work. Naturally, what we'll be putting in there is the whole file and not this patch, so things will be a bit different in final implementation.

The patch got merged, so someone needs to get that in if we don't get a new release. @tsimonq2 do you know if they plan on doing an LXQt release before, say, FeatureFreeze?

The good news: we've got the lxqt-globalkeys patch in proposed.

Since it's done, we don't have to then put the config file here right? It should go to rLUBUNTUDEFAULTSETTINGS ?