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.

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

#2433 Changed the XML namespaces to please jpm sign #2434

Merged
merged 5 commits into from
Mar 10, 2017

Conversation

andreicristianpetcu
Copy link
Contributor

No description provided.

@gorhill
Copy link
Owner

gorhill commented Mar 9, 2017

Q: did you try to see if the resulting xpi can be successfully dropped into Pale Moon, Sea Monkey?

@andreicristianpetcu
Copy link
Contributor Author

andreicristianpetcu commented Mar 9, 2017

Yes. This is what I ran:

Firefox - legacy

./tools/make-firefox.sh
# vim dist/build/uBlock0.firefox/install.rdf to edit a new ID in install.rdf since I don't own your addon and I cannot sign it
jpm xpi --addon-dir dist/build/uBlock0.firefox/
jpm sign --api-key $AMO_API_KEY --api-secret $AMO_API_SECRET --xpi dist/build/uBlock0.firefox/null.xpi

Firefox - webextension

./tools/make-webext.sh
# vim dist/build/uBlock0.webext/install.rdf to edit a new ID in install.rdf since I don't own your addon and I cannot sign it 
jpm xpi --addon-dir dist/build/uBlock0.webext/
jpm sign --api-key $AMO_API_KEY --api-secret $AMO_API_SECRET --xpi dist/build/uBlock0.firefox/null.xpi

I tested the result legacy addon from ublock_origin-1.11.3b1-an+fx+sm+tb.xpi in Pale Moon 27.1.2 (64-bit), Tor 6.5 (based on Mozilla Firefox 45.7.0), Tor 6.5.1 (based on Mozilla Firefox 45.8.0), GNU IceCat v45.7.0, Firefox Nightly 55.0a1 (2017-03-09) (64-bit). I'm sorry I do not have Sea Monkey installed and the installation failed on my Fedora for some reason. But here are the two generated builds so you can test.
Here is the legacy and the embeded WebExtension. There is the old Seamonkey metadata in the install.rdf so I guess it still works. Sorry for not testing it on this browser.
Hope all is fine :)

@andreicristianpetcu
Copy link
Contributor Author

HA! Made Seamonkey work! There is some strange issue with the button in the preferences on all browsers. I think this might be due to the addon ID change. Do you have any clue what might be the cause of this?

screenshot from 2017-03-10 01-40-04

@xofe
Copy link

xofe commented Mar 10, 2017

There is some strange issue with the button in the preferences on all browsers. I think this might be due to the addon ID change.

The ID is hard-coded in platform/firefox/vapi-background.js (search for optionsObserver) so for a custom build, you'll need to change it there too.

@andreicristianpetcu
Copy link
Contributor Author

I don't want a custom build with this fix (I have a bookmar pointing to the uBloc Origin settings page). I just wanted to make sure I did not break anything :D

@gorhill gorhill merged commit e8375f9 into gorhill:master Mar 10, 2017
@gorhill
Copy link
Owner

gorhill commented Mar 10, 2017

Thanks your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants