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

Option to disable cache (registry.txt) file #78

Merged
merged 3 commits into from
Oct 10, 2017
Merged

Conversation

JPvRiel
Copy link
Contributor

@JPvRiel JPvRiel commented Sep 16, 2017

Tested:

  • When disabled is set, no new entries are created in registry.txt
  • When disabled is not set, entries are created in registry.txt

There are some subtle quirks on edge cases, but otherwise functional:

  • Old stale entries in registry.txt will remain (disabling does not delete registry file)
  • When deleting an old/stale item stored before disabling, if the item was previously in registry.txt, the delete will only remove it from memory, but not from registry.txt
  • If a registry file previously existed, and was disabled, and then is re-enabled, the previous registry file is overwritten and replaced with the current clipboard content.
  • If gnome-shell is restarted, the old stale registry.txt file gets loaded (this behaviour is probably undesirable)

A future tweak might be to explicitly trigger deleting the registry file when toggled to disable in order to clear up the quirks. For now, if users really want this feature, suggest deleting the registry.txt file manually.

- related to issue Tudmotu#58
- currently not working due to a dconf schema install problem
- related to issue Tudmotu#58
- currently not working due to a dconf schema install problem
- fixed gap/missing label number
- changed bundle script to have a more cross-platoform shabang
- tweaked description for option
@JPvRiel
Copy link
Contributor Author

JPvRiel commented Oct 8, 2017

Just a nudge here to see if this looks any good for a merge?

@Tudmotu Tudmotu merged commit 85cb57a into Tudmotu:master Oct 10, 2017
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.

None yet

2 participants