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
Evolution ews: wrap evolution and evolution data server, fix nixos module. #103135
Conversation
ah yes indeed, sorry :° |
pkgs/top-level/all-packages.nix
Outdated
@@ -20208,6 +20208,8 @@ in | |||
inherit (gnome3) evince; | |||
evolution-data-server = gnome3.evolution-data-server; | |||
evolution-ews = callPackage ../applications/networking/mailreaders/evolution/evolution-ews { }; | |||
evolution = callPackage ../applications/networking/mailreaders/evolution/evolution { }; | |||
evolutionBundle = callPackage ../applications/networking/mailreaders/evolution/evolution/wrapper.nix { }; |
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.
This is non-standard naming that is not used anywhere. The -with-plugins
is at least almost an idiom.
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.
If we are to talk about naming: the problem is that this is rather an evolution-data-server-with-plugins. Where evolution somehow counts as a plugin. an evolutionWithPlugins which can possibly not contain evolution but only evolution-data-server would be somewhat strange. So I welcome better naming suggestions which can convey that.
nixos/modules/services/desktops/gnome3/evolution-data-server.nix
Outdated
Show resolved
Hide resolved
I solved the merge conflict. |
I solved anther merge conflict. I think the PR is ready now, and I've been using it successfully for some time now. |
I tried running |
solved the merge conflict. |
I tested again, the evolution-with-plugins binary and it seems we are back to the problem described here: #92775 (comment) |
@dasj19 : you must ensure that you have installed no other software depending on evolution-data-server. This includes gnome-calendar for example. Also, because evolution-data-server has long running processes launched with Edit sorry dasj for the wrong ping. |
I have been using this nearly daily for months now without a problem. |
@symphorien I can not get it to work, I have installed evolutionWithPlugins and did a reboot afterwards. When I open evolution I can add an ews account but it does not show up in the list with accounts. I do not know what else to try... |
I've tested this PR on top of
The setup of the O365 account was somewhat complicated due to tenant and application ids etc, but that is part of EWS. Thanks for the effort. Really helpful. |
@tolbrino Yes, it works with the options you provide. Thanks. |
Mmh enabling |
This might just be on my end because GPG is used. |
I tried to upstream the patch to evolution, and upstream came up with a better solution. A simple wrapper with an environment variable without symlinkjoin will be available starting next release: https://gitlab.gnome.org/GNOME/evolution/-/merge_requests/77#note_1044591 |
Motivation for this change
Extends and replaces #92775
cc @dasJ @jtojnar
Things done
I've been using this for several weeks.
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)