The default bug view has changed. See this FAQ.

Disable addon signing via comm-central mozconfigs

RESOLVED FIXED in Thunderbird 43.0

Status

Thunderbird
Build Config
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

39 Branch
Thunderbird 43.0

Thunderbird Tracking Flags

(thunderbird41+)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

2 years ago
bug 1186522 will add add-on signing to mozconfig.common, but we want to disable it. We need to do this in some file that includes build/mozconfig.common, right now this is each platform mozconfig.

I think we could solve this by creating mail/configs/mozconfigs/mozconfig.common, which includes /build/mozconfig.common and then can be included from our platform specific mozconfigs. This would also allow us to add things like --enable-calendar there.

Maybe there are other options though. jcranmer?
Flags: needinfo?(Pidgeot18)
We basically can't touch anything in build/*, since that's effectively identical to mozilla-central (mozconfigs are REALLY wonky) there. Making a common mail mozconfig base might be a good idea.
Flags: needinfo?(Pidgeot18)

Comment 2

2 years ago
I've been wondering about this.

Say we get the nightly from mail/config/mozconfigs/*platform*.

It's my understanding that |. mail/configs/mozconfigs/mozconfig.common| won't
work with our current build config (and probably by extension, relative
paths won't work as well).

so would it make sense to have some sort of fixed topsrcdir set? (kinda
cheating I know:  i.e.   if "mozilla" in topsrcdir,  use relative paths 
(../mail/...)  otherwise, use the actual path  i.e. mail/...
(Assignee)

Updated

2 years ago
Depends on: 1195143
(Assignee)

Updated

2 years ago
Depends on: 1195462
(Assignee)

Updated

2 years ago
Depends on: 1195468
(Assignee)

Comment 3

2 years ago
Created attachment 8648902 [details] [diff] [review]
Fix - v1

ewong, yes, there needs to be some changes to the mozconfigs to make it work like you suggested. I've done this in bug 1195468. This patch requires all dependent bugs to be applied/pushed.

I hope disabling everything is the right way to go about it. Maybe that will have the nice side-effect of fixing tests that fail because of addon-signing being enabled.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #8648902 - Flags: review?(Pidgeot18)

Comment 4

2 years ago
This is needed prior to a beta release IIUC
tracking-thunderbird41: --- → +
(Assignee)

Comment 5

2 years ago
Created attachment 8658270 [details] [diff] [review]
Fix - v2

As we aren't doing the whole common mozconfig dance now, here is a patch that adds it to all mozconfigs. I've added it after including mozconfig.common because that is what jlund is doing in attachment 8656163 [details] [diff] [review].
Attachment #8648902 - Attachment is obsolete: true
Attachment #8648902 - Flags: review?(Pidgeot18)
Attachment #8658270 - Flags: review?(Pidgeot18)
(Assignee)

Updated

2 years ago
No longer depends on: 1195143, 1195462, 1195468
Attachment #8658270 - Flags: review?(Pidgeot18) → review+
(Assignee)

Comment 6

2 years ago
Pushed to comm-central changeset cfb6f301cc13
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 43.0
You need to log in before you can comment on or make changes to this bug.