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)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 77.0

People

(Reporter: darktrojan, Assigned: rjl)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

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.

I ran artifact and regular Linux64 builds locally (clobbered).
Attachment #9142606 - Flags: review?(geoff)
Assignee: nobody → rob
Status: NEW → ASSIGNED
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+

Yeah Python indenting for the most part.

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/d010d86cedeb
Fix artifact builds when --enable-openpgp is set. r=darktrojan DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 77.0
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
Attachment #9142715 - Flags: review?(mkmelin+mozilla) → review+
Keywords: regression
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: