Project

General

Profile

Feature #15298

Rebase our Thunderbird custom package on top of 52.6.0

Added by anonym almost 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Elevated
Assignee:
-
Category:
-
Target version:
Start date:
02/09/2018
Due date:
% Done:

100%

Feature Branch:
feature/15298-thunderbird-52.6.0
Type of work:
Code
Blueprint:
Starter:
Affected tool:
Email Client

Related issues

Blocks Tails - Feature #13245: Core work 2018Q1: Foundations Team Resolved 06/29/2017

Associated revisions

Revision 213919ea (diff)
Added by anonym almost 2 years ago

Enable the feature-15298-thunderbird-52.6.0 APT overlay.

Will-fix: #15298

Revision 51eb29fe
Added by intrigeri almost 2 years ago

Merge remote-tracking branch 'origin/feature/15298-thunderbird-52.6.0' into devel (Fix-committed: #15298)

History

#1 Updated by intrigeri almost 2 years ago

#2 Updated by intrigeri almost 2 years ago

  • Status changed from New to Confirmed

#3 Updated by intrigeri almost 2 years ago

  • Subject changed from Rebase our Thunderbird custom package on top of 52.6.0 to Rebase our Thunderbird custom package on top of 52.6.0

#4 Updated by intrigeri almost 2 years ago

Note that the Vcs-Git moved to Salsa, so our release process doc needs to be adjusted.

#5 Updated by anonym almost 2 years ago

  • Status changed from Confirmed to In Progress

#6 Updated by anonym almost 2 years ago

  • % Done changed from 0 to 40
  • Feature Branch set to feature/15298-thunderbird-52.6.0

intrigeri wrote:

Note that the Vcs-Git moved to Salsa, so our release process doc needs to be adjusted.

Thanks! Updated.

#7 Updated by anonym almost 2 years ago

  • % Done changed from 40 to 50

thunderbird.feature passes on my system! I was intending to use the same branch for #11973, but let's not block.

#8 Updated by anonym almost 2 years ago

  • Assignee changed from anonym to bertagaz
  • QA Check set to Ready for QA

#9 Updated by intrigeri almost 2 years ago

  • Assignee changed from bertagaz to intrigeri

(As per discussion with anonym, in order to make bertagaz' review plate lighter and increase the chances he can deal with what's left there in a timely manner.)

#10 Updated by intrigeri almost 2 years ago

  • % Done changed from 50 to 60

git diff debian/1%52.6.0-1_deb9u1..debian/52.6.0-1_deb9u1.0tails1 shows that we modify a Debian changelog entry (adding some whitespace). I guess that's a typo. No idea when it was introduced, not a blocker, but please fix that in our icedove.git.

Other than that, code review passes!

#11 Updated by anonym almost 2 years ago

  • % Done changed from 60 to 50

intrigeri wrote:

git diff debian/1%52.6.0-1_deb9u1..debian/52.6.0-1_deb9u1.0tails1 shows that we modify a Debian changelog entry (adding some whitespace). I guess that's a typo. No idea when it was introduced, not a blocker, but please fix that in our icedove.git.

Crap, it was done by mistake in one of my merge conflict resolutions. Now fixed, thanks!

#12 Updated by intrigeri almost 2 years ago

  • % Done changed from 50 to 60

I should have a full test suite run done within 3.5 hours so expect a merge either late tonight of early tomorrow, if everything goes well :)

#13 Updated by intrigeri almost 2 years ago

  • Status changed from In Progress to 11
  • % Done changed from 60 to 100

#14 Updated by intrigeri almost 2 years ago

  • Assignee deleted (intrigeri)
  • QA Check changed from Ready for QA to Pass

#15 Updated by bertagaz almost 2 years ago

  • Status changed from 11 to Resolved

Also available in: Atom PDF