Build/Include the OpenPGP code by default in comm-central
Categories
(MailNews Core :: Security: OpenPGP, enhancement)
Tracking
(Not tracked)
People
(Reporter: KaiE, Assigned: KaiE)
References
(Depends on 1 open bug, Regressed 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
text/x-phabricator-request
|
Details |
The OpenPGP code should soon be ready for initial testing in nightly/daily Thunderbird builds. To enable it, we can flip the build pref.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
This needs to land at the same time as bug 1625135, to ensure existing tests will continue to pass.
To actually being able to use the code, we need the RNP library and dependencies, as tracked in bug 1621785.
However, it's not a strict dependency at this time, because with the changes from bug 1625574, we should be able to fail gracefull, and non-OpenPGP code still runs.
Assignee | ||
Comment 2•5 years ago
|
||
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
We need the patch "Remove old migration code and setup wizard" from bug 1631019 before enabling by default.
Without it, we get a test failure, with the symptom described in bug 1631019 comment 3.
Assignee | ||
Comment 4•5 years ago
|
||
(In reply to Kai Engert (:KaiE:) from comment #3)
We need the patch "Remove old migration code and setup wizard" from bug 1631019 before enabling by default.
Without it, we get a test failure, with the symptom described in bug 1631019 comment 3.
That patch is a little bigger and removes a lot of code.
If we don't get a review from Patrick today, alternatively we could land this minimal patch.
Comment 5•5 years ago
|
||
I'll review the patch this afternoon (ca. 17:30).
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 6•5 years ago
|
||
Phabricator has a limitation, it allows neither Rob nor me to mark the patch as reviewed.
However, my initial patch was a one line obvious thing. And Rob did the real improvement patch.
r=kaie on Rob's improvement, and r=rjl (from Matrix) on my initial line to enable it.
Pushed by kaie@kuix.de:
https://hg.mozilla.org/comm-central/rev/f0676c4790a1
Build/Include the OpenPGP code by default in comm-central. r=kaie,r=rjl
Updated•5 years ago
|
Description
•