Cleanup config initialization, add local config options#5452
Merged
droidmonkey merged 1 commit intorelease/2.6.2from Sep 26, 2020
Merged
Cleanup config initialization, add local config options#5452droidmonkey merged 1 commit intorelease/2.6.2from
droidmonkey merged 1 commit intorelease/2.6.2from
Conversation
* Fix #5313, allow specifying local config path using environment variable and command line flag * Add command line flag `--localconfig <path>` to specify a file path to use for the local configuration settings. * Add environment variable support to set config files paths: `KPXC_CONFIG` and `KPXC_CONFIG_LOCAL` to override default locations. * Reorder startup sequence to load specified config files earlier to allow for theme settings and other early options to be picked up. * Removed old command line option `--pw`, no longer used. * Attempt a fix of application not closing when last window is gone. Only set `QApplication::setQuitOnLastWindowClosed(true)` when tray icon is enabled instead of always.
0465676 to
480cbf4
Compare
|
When using variable |
Member
Author
|
Set it to whatever you want, but this points to a file not a directory. |
|
Oh... So I guess it should be |
Member
Author
|
Give it a try, test it out please |
|
OK. I tried to build keepassxc following readme (with cmake arguments Given that I'm not interested in building and debugging keepassxc manually I'll just wait until this get merged to master and packaged by my linux distribution maintainers. |
phoerious
approved these changes
Sep 25, 2020
Member
QSettings creates directories if they don't exist. |
phoerious
added a commit
that referenced
this pull request
Oct 21, 2020
Added - Add option to keep window always on top to view menu [#5542] - Move show/hide usernames and passwords to view menu [#5542] - Add command line options and environment variables for changing the config locations [#5452] - Include TOTP settings in CSV import/export and add support for ISO datetimes [#5346] Changed - Mask sensitive information in command execution confirmation prompt [#5542] - SSH Agent: Avoid shortcut conflict on macOS by changing "Add key" to Ctrl+H on all platforms [#5484] Fixed - Prevent data loss with drag and drop between databases [#5536] - Fix crash when toggling Capslock rapidly [#5545] - Don't mark URL references as invalid URL [#5380] - Reset entry preview after search [#5483] - Set Qt::Dialog flag on database open dialog [#5356] - Fix sorting of database report columns [#5426] - Fix IfDevice matching logic [#5344] - Fix layout issues and a stray scrollbar appearing on top of the entry edit screen [#5424] - Fix tabbing into the notes field [#5424] - Fix password generator ignoring settings on load [#5340] - Restore natural entry sort order on application load [#5438] - Fix paperclip and TOTP columns not saving state [#5327] - Enforce fixed password font in entry preview [#5454] - Add scrollbar when new database wizard exceeds screen size [#5560] - Do not mark database as modified when viewing Auto-Type associations [#5542] - CLI: Fix two heap-use-after-free crashes [#5368,#5470] - Browser: Fix key exchange not working with multiple simultaneous users on Windows [#5485] - Browser: Fix entry retrieval when "only best matching" is enabled [#5316] - Browser: Ignore recycle bin on KeePassHTTP migration [#5481] - KeeShare: Fix import crash [#5542] - macOS: Fix toolbar theming and breadcrumb display issues [#5482] - macOS: Fix file dialog randomly closing [#5479] - macOS: Fix being unable to select OPVault files for import [#5341]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix Local config data is cleared when ~/.cache mounted on tempfs #5313, allow specifying local config path using environment variable and command line flag
Add command line flag
--localconfig <path>to specify a file path to use for the local configuration settings.Add environment variable support to set config files paths:
KPXC_CONFIGandKPXC_CONFIG_LOCALto override default locations.Reorder startup sequence to load specified config files earlier to allow for theme settings and other early options to be picked up.
Removed old command line option
--pw, no longer used.Attempt a fix of application not closing when last window is gone. Only set
QApplication::setQuitOnLastWindowClosed(true)when tray icon is enabled instead of always.Replacement for #5325
Screenshots
Testing strategy
Tested on Windows
Type of change