Skip to content

kmymoney: add gappsWrapperArgs to qtWrapperArgs#509718

Open
das-g wants to merge 1 commit into
NixOS:masterfrom
das-g:kmymoney-gappsWrapperArgs
Open

kmymoney: add gappsWrapperArgs to qtWrapperArgs#509718
das-g wants to merge 1 commit into
NixOS:masterfrom
das-g:kmymoney-gappsWrapperArgs

Conversation

@das-g

@das-g das-g commented Apr 13, 2026

Copy link
Copy Markdown
Member

Add gappsWrapperArgs to KMyMoney's qtWrapperArgs, so that attempting to export a report (as well as presumably any other action requiring a file chooser dialog) will work again also on GNOME, rather than crashing with

(kmymoney:26393): GLib-GIO-ERROR **: 16:18:43.780: Settings schema ‘org.gtk.Settings.FileChooser’ is not installed

This change was authored following the example of #507382 as of 1e50738, though with the addition of TODO comments to remind me to revert this once the underlying problem has been solved globally in nixpkgs, e.g. by #507455 or #271037.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

so that attempting to export a report (as well as presumably any other
action requiring a file chooser dialog) will work again also on GNOME,
rather than crashing with
	(kmymoney:26393): GLib-GIO-ERROR **: 16:18:43.780: Settings schema ‘org.gtk.Settings.FileChooser’ is not installed

This change was authored following the example of
NixOS#507382
as of 1e50738,
though with the addition of TODO comments to remind me to revert this
once the underlying problem has been solved globally in nixpkgs.
@das-g das-g force-pushed the kmymoney-gappsWrapperArgs branch from da5b5b2 to 72abb27 Compare April 13, 2026 20:43
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Apr 13, 2026
@das-g das-g requested review from chillcicada and twhitehead April 13, 2026 20:54
@nixpkgs-ci nixpkgs-ci Bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Apr 13, 2026
@nixpkgs-ci nixpkgs-ci Bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Apr 14, 2026
@das-g

das-g commented Apr 14, 2026

Copy link
Copy Markdown
Member Author

After hours of local compiling, I was able to confirm that the changes of PR #507455 rebased onto master (das-g/nixpkgs@0b80b76...42b1161) do indeed solve this problem for KMyMoney, too. (Based on staging as that PR currently is (and should be), it probably would have been even more hours of local compiling.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants