Skip to content
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, thunderbird-bin: 45.8.0 -> 52.0 #24623

Closed
wants to merge 2 commits into from

Conversation

taku0
Copy link
Contributor

@taku0 taku0 commented Apr 4, 2017

Motivation for this change

Various enhancements and changes.

“IMPORTANT: The way images are included in a compose window has changed. Images are now included as data URIs and not as references to parts of other messages or operating system files. This allows better interoperability with office packages such as MS Office or LibreOffice. Images linked from locations on the internet will no longer be downloaded and attached to the message automatically. This can be changed for each image individually via the Image Properties dialog or globally by setting the preference mail.compose.attach_http_images.”

https://www.mozilla.org/en-US/thunderbird/52.0/releasenotes/

Related commits: 90f5be3

Note that the calendar add-on (Lightning) does not seem to be compatible with 52 yet.

https://addons.mozilla.org/en-US/thunderbird/addon/lightning/

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@taku0, thanks for your PR! By analyzing the history of the files in this pull request, we identified @garbas, @dezgeg and @vcunat to be potential reviewers.

wrapProgram "$out/bin/thunderbird" \
--argv0 "$out/bin/.thunderbird-wrapped" \
--prefix XDG_DATA_DIRS : "$GSETTINGS_SCHEMAS_PATH:" \
--suffix XDG_DATA_DIRS : "$XDG_ICON_DIRS"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose wrapGAppsHook can be also used here, right @abbradar?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct!

@@ -116,6 +119,8 @@ stdenv.mkDerivation {
stdenv.cc.cc
];

buildInputs = [ makeWrapper gtk3 defaultIconTheme ];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makeWrapper should go to nativeBuildInputs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed. I will also fix firefox-bin after this PR is merged.

@rasendubi
Copy link
Member

Passed nox-review locally on NixOS with sandboxing enabled.

@abbradar
Copy link
Member

abbradar commented Apr 6, 2017

I'm trying to solve a crash in Thunderbird so I can merge this along with my fixes (most likely we need to disable system cairo). I'll also see if we can enable optional GTK3 support.

@aristidb
Copy link
Contributor

aristidb commented Apr 8, 2017

@abbradar Is the crash in Thunderbird caused by this change?

@abbradar
Copy link
Member

abbradar commented Apr 8, 2017

@aristidb Nope, it's a separate problem, unrelated to this -- I'm just testing things in bulk. I still haven't solved it (plan to finish today or tomorrow) so if anybody wants he/she can merge this earlier.

@abbradar
Copy link
Member

abbradar commented Apr 9, 2017

I've merged it as 55ff94b and 77f0ad5. Along with b023370 this finally fixes crashes that I've experienced for many months now. Thank you!

@abbradar abbradar closed this Apr 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants