Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 388647 - mozextension.eclass: Add new function for installing extensions in new standard install location
Summary: mozextension.eclass: Add new function for installing extensions in new standa...
Status: UNCONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Mozilla Gentoo Team
URL: https://developer.mozilla.org/en/Inst...
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2011-10-27 09:02 UTC by Shubhadeep Chaudhuri
Modified: 2012-05-23 18:22 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
mozextension.eclass.patch (mozextension.eclass.patch,738 bytes, patch)
2011-10-27 09:02 UTC, Shubhadeep Chaudhuri
Details | Diff
mozextension.eclass.patch (mozextension.eclass.patch,738 bytes, patch)
2011-10-27 10:56 UTC, Shubhadeep Chaudhuri
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Shubhadeep Chaudhuri 2011-10-27 09:02:48 UTC
Created attachment 290989 [details, diff]
mozextension.eclass.patch

The mozextenions eclass hasn't been updated since 2007. The xpi_install function must be changed or a new install function must be added to the eclass to utilize the new standard install location for extensions which is /usr/lib/mozilla/extensions/<appid>. 

This feature has been implemented a long time back and is documented here:
https://developer.mozilla.org/en/Installing_extensions

Mozilla bug entry requesting the feature:
https://bugzilla.mozilla.org/show_bug.cgi?id=311008

Attached a patch which modifies xpi_install to do just this.
Comment 1 Shubhadeep Chaudhuri 2011-10-27 10:56:37 UTC
Created attachment 291003 [details, diff]
mozextension.eclass.patch

Corrected patch
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2011-10-27 17:52:21 UTC
eclass lacks a maintainer tag.
Comment 3 Jory A. Pratt gentoo-dev 2011-10-31 12:29:38 UTC
Actually the patch is incorrect, appid does not need to be included it is the emid that is used to install. I will have to double check this to make sure we do not break enigmail before I am comfortable committing this to the tree.