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
Thunderbird: Fix default mail client detection #24961
Conversation
The refactoring in b023370 ported changes from firefox-unwrpped, but dropped features in firefox's wrapper. Add the desktop item and remove useless dev files.
89bcc49
to
c11c9ff
Compare
Ooops, found a bug in the meantime. it would be simpler to enforce gtk3 to get only one build to test. |
c11c9ff
to
ac34cc1
Compare
Voilà, I got it working properly. This is difficult because we need to use wrapGAppsHook when gtk3 is enabled, and makeWrapper otherwise... @abbradar Would you mind reviewing this ? |
ac34cc1
to
eb8f604
Compare
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.
From the bird's view the fix seems hacky to me. However this may very well be the best way to do this and I have a very little time to confirm. So, unless I'll have some in several days I'm okay with this merged as is.
It is hacky because
|
Tested; it works for me. Thanks for this and for restoring features which I accidentially removed! |
Thanks for reviewing :-). |
Motivation for this change
fix regression from b023370, and default mail reader detection (see also #23516)
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)/cc @abbradar