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?
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.
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/...
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)
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 #8658270 - Flags: review?(Pidgeot18) → review+
Pushed to comm-central changeset cfb6f301cc13
Status: ASSIGNED → RESOLVED
Closed: 4 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.