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
ibus: fix dconf database #85992
ibus: fix dconf database #85992
Conversation
Is bd6c12b#diff-a149871f918ec698c20b8b19f3f2e6d6 no longer necessary? |
Otherwise it still look reasonable to me at a glance but did not test it. |
Fixes this warning at ibus-daemon startup: (ibus-dconf:15691): dconf-WARNING **: 21:49:24.018: unable to open file '/etc/dconf/db/ibus': Failed to open file ?/etc/dconf/db/ibus?: open() failed: No such file or directory; expect degraded performance
This reverts commit ee5cba2.
Oh, indeed I lost some commits with git. I will fix this. |
This passes the ibus and xfce test, and I checked in a vm that ibus-daemon does not warn about dconf anymore. |
concatStringsSep "\n" ( | ||
mapAttrsToList ( | ||
name: path: '' | ||
ln -s ${path} $out/profile/${name} | ||
'' | ||
) cfg.profiles | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why won't the profiles already be linked by symlinkJoin
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was also confused by this but profiles are not packages, they are just file paths.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, the symlinkJoin is only for the option packages
and this loop is for the option profiles
. So I think this is correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand, thanks!
concatStringsSep "\n" ( | ||
mapAttrsToList ( | ||
name: path: '' | ||
ln -s ${path} $out/profile/${name} | ||
'' | ||
) cfg.profiles | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand, thanks!
Motivation for this change
#85892 was reverted because of an evaluation error which manifested among other in xfce test.
This is a fixed PR where the xfce test evaluates.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)