Closed
Bug 1632343
Opened 5 years ago
Closed 5 years ago
--enable-openpgp is incompatible with --enable-artifact-builds
Categories
(MailNews Core :: Security: OpenPGP, defect)
MailNews Core
Security: OpenPGP
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 77.0
People
(Reporter: darktrojan, Assigned: rjl)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
10.78 KB,
patch
|
darktrojan
:
review+
|
Details | Diff | Splinter Review |
2.40 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
The OpenPGP configure checks require a compiler. Artifact builds require not having a compiler. This isn't going to work.
For now, this can be worked around with --disable-openpgp
, so probably what needs to happen is a check that we're not in an artifact build before including openpgp.configure.
Assignee | ||
Comment 1•5 years ago
|
||
I ran artifact and regular Linux64 builds locally (clobbered).
Attachment #9142606 -
Flags: review?(geoff)
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → rob
Status: NEW → ASSIGNED
Reporter | ||
Comment 2•5 years ago
|
||
Comment on attachment 9142606 [details] [diff] [review]
Fix artifact builds when --enable-openpgp is set
Assuming it's just white space causing most of the change, this is fine.
Attachment #9142606 -
Flags: review?(geoff) → review+
Assignee | ||
Comment 3•5 years ago
|
||
Yeah Python indenting for the most part.
Keywords: checkin-needed-tb
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/d010d86cedeb
Fix artifact builds when --enable-openpgp is set. r=darktrojan DONTBUILD
Updated•5 years ago
|
Target Milestone: --- → Thunderbird 77.0
Assignee | ||
Comment 5•5 years ago
|
||
COMPILE_ENVIRONMENT will not be set for L10n repacks or artifact
builds.
Attachment #9142715 -
Flags: review?(mkmelin+mozilla)
Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/comm-central/rev/53c02cf46815
Follow-up: Fix l10n repacks on Daily build. r=mkmelin rs=bustage-fix DONTBUILD
Updated•5 years ago
|
Attachment #9142715 -
Flags: review?(mkmelin+mozilla) → review+
Updated•4 years ago
|
Keywords: regression
You need to log in
before you can comment on or make changes to this bug.
Description
•