Closed
Bug 1075048
Opened 10 years ago
Closed 10 years ago
During Build, lightning is installed in wrong directory
Categories
(Calendar :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
3.8
People
(Reporter: Taraman, Assigned: Fallen)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Fallen
:
review+
|
Details | Diff | Splinter Review |
When building TB with "enable-calendar" on linux (I can't test other platforms at the moment), the extension is installed in "objdir/dist/bin/distribution/extension" ISO "objdir/dist/bin/extension"
Thus the addon is not enabled.
Comment 1•10 years ago
|
||
From my build on Windows 7 yesterday: Lightning is copied to obj-i686-pc-mingw32\dist\bin\distribution\extensions\{e2fda1a4-762b-4020-b5ad-a41df1933103}
When I start Thunderbird by calling |mach run| Lightning is installed and available. Probably because it was automatically deployed into the new profile at obj-i686-pc-mingw32\tmp\scratch_user\extensions\{e2fda1a4-762b-4020-b5ad-a41df1933103}
Reading e.g. http://mike.kaply.com/2012/02/09/integrating-add-ons-into-firefox/ this seems the correct way to deploy extensions automatically for users.
Maybe you are using an existing profile and therefore need to install Lightning manually?
Assignee | ||
Comment 2•10 years ago
|
||
I bet this is something about the mac os signing changes. Previously just running it would be enough, at least for development mode I think we shouldn't require any more extra steps. Its probably worth asking folks in #build about this or CC'ing someone.
Reporter | ||
Comment 3•10 years ago
|
||
Stefan, you are right. I use an existing profile.
But until recently, the extension was installed into ../dist/bin/extension and showed up correctly. If I copy it there manually, this still works.
Assignee | ||
Comment 4•10 years ago
|
||
This should take care. It reverts the behavior so that the extension is installed back into dist/bin, and makes sure that the rsync for mac drops it in Contents/Resources.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #8509609 -
Flags: review?(bugspam.Callek)
Comment 5•10 years ago
|
||
Comment on attachment 8509609 [details] [diff] [review]
Fix - v1
Review of attachment 8509609 [details] [diff] [review]:
-----------------------------------------------------------------
I'm not great on the lightning makefile stuff, but looks correct
Attachment #8509609 -
Flags: review?(bugspam.Callek) → review+
Comment 6•10 years ago
|
||
Great, with this patch I managed to successfully run calendar xpcshell tests for local Windows build again.
Assignee | ||
Comment 8•10 years ago
|
||
I was just missing the checkin comment, sorry for the delay.
Attachment #8509609 -
Attachment is obsolete: true
Flags: needinfo?(philipp)
Attachment #8518121 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 9•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 3.8
You need to log in
before you can comment on or make changes to this bug.
Description
•