Project

General

Profile

Bug #16834

Upgrade Thunderbird to 60.7.2+

Added by intrigeri about 1 month ago. Updated 12 days ago.

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
-
Target version:
Start date:
Due date:
% Done:

100%

Feature Branch:
https://salsa.debian.org/tails-team/tails/merge_requests/31
Type of work:
Code
Blueprint:
Starter:
Affected tool:
Email Client


Related issues

Blocks Tails - Feature #16209: Core work: Foundations Team Confirmed 03/22/2019

Associated revisions

Revision b4308410 (diff)
Added by intrigeri 16 days ago

Enable the bugfix-16834-thunderbird-60.7.2-force-all-tests APT overlay (refs: #16834).

Revision f9142ca6 (diff)
Added by intrigeri 16 days ago

Patch Thunderbird packages from Debian when building Tails images (refs: #16834, #6156).

This avoids the need to build and upload Thunderbird packages every now and
then. Instead, we'll need to refresh our patches when they don't apply
as-is anymore.

Current patchset imported from our icedove.git at
commit a3eab1e85558f2c9cf01d332920ac81fea3cd9fd.

Revision 8b217f0b
Added by segfault 15 days ago

Merge branch 'bugfix/16834-thunderbird-60.7.2+force-all-tests' into stable (Fix-committed: #16834)

History

#1 Updated by intrigeri about 1 month ago

#2 Updated by intrigeri about 1 month ago

  • Description updated (diff)

#3 Updated by intrigeri 30 days ago

  • Priority changed from Normal to Elevated

#4 Updated by intrigeri 19 days ago

  • Assignee set to intrigeri

@segfault, I'm taking this one as a fallback but I'll wait until Saturday to work on it, hoping you can take it :)

#5 Updated by intrigeri 16 days ago

  • Status changed from Confirmed to In Progress

#6 Updated by intrigeri 16 days ago

  • Feature Branch set to bugfix/16834-thunderbird-60.7.2+force-all-tests

#7 Updated by intrigeri 16 days ago

Tested manually with an email address whose ISP is not pre-configured in Thunderbird itself (see #16856). features/thunderbird.feature passes locally. My changes may break reproducibility so I'll check that before sending this for QA, but a first code review could already happen and save us time later.

#8 Updated by intrigeri 16 days ago

  • Status changed from In Progress to Needs Validation

(So that reproducibility is tested on Jenkins.)

#9 Updated by intrigeri 16 days ago

  • Assignee changed from intrigeri to segfault

@segfault, there you go :)

#10 Updated by segfault 15 days ago

  • Status changed from Needs Validation to In Progress
  • Assignee changed from segfault to intrigeri
  • Feature Branch changed from bugfix/16834-thunderbird-60.7.2+force-all-tests to https://salsa.debian.org/tails-team/tails/merge_requests/31

I like the change! Just have a few comments, wrote them on the gitlab MR I created.

#11 Updated by intrigeri 15 days ago

I like the change!

:)

Just have a few comments, wrote them on the gitlab MR I created.

@segfault, thanks!

I'll try my best but it's not very likely that I manage to look into that before Monday night, and it'll be too late for 3.15. So if you think that the state of my branch (including the Thunderbird security update) is a net improvement over not merging it at all, please merge it and I'll handle your review post-3.15.

#12 Updated by intrigeri 15 days ago

  • Status changed from In Progress to Needs Validation
  • Assignee changed from intrigeri to segfault

@segfault, I've replied on the MR, fixed one issue, and the only remaining discussion is about a possible follow-up step we could take, even further on the path I've taken here. It does not look like a blocker for merging, right?

#13 Updated by segfault 15 days ago

  • Status changed from Needs Validation to Fix committed
  • % Done changed from 0 to 100

#14 Updated by CyrilBrulebois 12 days ago

  • Status changed from Fix committed to Resolved

Also available in: Atom PDF