Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fresh installation of RP on Firefox 4.0b7+ broken #142

Closed
jsamuel opened this issue Dec 22, 2011 · 3 comments
Closed

fresh installation of RP on Firefox 4.0b7+ broken #142

jsamuel opened this issue Dec 22, 2011 · 3 comments
Labels
bug report

Comments

@jsamuel
Copy link
Member

jsamuel commented Dec 22, 2011 

imported trac ticket
created: 2010-11-05 11:45:15
reporter: justin

It appears that installing RP for the first time on Fx 4.0b7+ results in a non-working RP. There's an error about the use of __LOCATION__. Probably related, Fx now keeps the xpi as an archive on disk, not extracted. This may explain why an existing RP install continues working after upgrading Fx: the xpi is extracted and __LOCATION__ works.

The full error:

{{{
Error: LOCATION is not defined
Source File: jar:file:///path/to/.mozilla/firefox/asfdsafd.profilename/extensions/requestpolicy@requestpolicy.com.xpi!/components/requestpolicyService.js
Line: 432
}}}
@jsamuel
Copy link
Member Author

jsamuel commented Dec 22, 2011 

imported trac comment
created: 2010-11-05 11:51:24
author: justin

Forgot to say: Thanks to Aleksej for the report.

@jsamuel
Copy link
Member Author

jsamuel commented Dec 22, 2011 

imported trac comment
created: 2010-11-06 15:21:04
author: justin

Fixed in r361. I have a feeling the reason I didn't use this solution before was probably because I wasn't adding the correct resource line to chrome.manifest when I had tried. I probably was using a jar: reference even though the modules directory isn't in requestpolicy.jar (which only includes content/, locale/ and skin/).

@jsamuel
Copy link
Member Author

jsamuel commented Dec 22, 2011 

imported trac comment
created: 2010-11-07 11:52:52
author: justin

Thanks also to [http://forums.mozillazine.org/viewtopic.php?p=10075793#p10075793 aabbqrstp] for the initial report.

@jsamuel jsamuel closed this as completed Dec 22, 2011
@msxfm msxfm mentioned this issue Jul 7, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug report
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jsamuel