Closed Bug 1102013 Opened 6 years ago Closed 6 years ago

mail/installer/package-manifest.in needs to be updated after the cleanup in bug 1096494

Categories

(Thunderbird :: Installer, defect)

defect
Not set
normal

Tracking

(thunderbird36 fixed)

RESOLVED FIXED
Thunderbird 36.0
Tracking Status
thunderbird36 --- fixed

People

(Reporter: spohl, Assigned: spohl)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1096494 +++

Unfortunately, I didn't realize that the patch in bug 1096494 would bust comm-central on OSX. All that seems to be necessary is for mail/installer/package-manifest.in to use @RESPATH@ instead of @BINPATH@ for the resources that are located under Contents/Resources. The patch in bug 1096494 should be a good illustration how this was done for Firefox.

Josiah, since you've handled most of the v2 signing changes for TB, is this something that you could tackle? Happy to assist with feedback or anything else that would be helpful.
Yes, that's fine. Thanks for filing the bug!
Assignee: nobody → josiah
Status: NEW → ASSIGNED
Attached patch PatchSplinter Review
Thanks, Josiah! I actually found some time this morning to write the patch myself. I hope you didn't get started yet. Thanks for offering your help though!
Assignee: josiah → spohl.mozilla.bugs
Attachment #8526098 - Flags: review?(Pidgeot18)
Comment on attachment 8526098 [details] [diff] [review]
Patch

Review of attachment 8526098 [details] [diff] [review]:
-----------------------------------------------------------------

Ugh, I wish packaging were more sane. I pushed this to try and it passed tests at least as well as anything else these days, and I don't see anything that stands out as weird or wrong, so r+.
Attachment #8526098 - Flags: review?(Pidgeot18) → review+
Keywords: checkin-needed
Not sure how approvals etc. work on comm-central, and it's getting too late here to watch the tree. Setting checkin-needed in the hope that this keyword applies to comm-central too.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 36.0
You need to log in before you can comment on or make changes to this bug.