[MIR] webapps packages

Bug #1053463 reported by Michael Terry
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
polib (Ubuntu)
Fix Released
Undecided
Unassigned
Quantal
Fix Released
Undecided
Unassigned
unity-firefox-extension (Ubuntu)
Fix Released
Undecided
Unassigned
Quantal
Fix Released
Undecided
Unassigned
webapps-applications (Ubuntu)
Fix Released
Undecided
Unassigned
Quantal
Fix Released
Undecided
Unassigned
webapps-greasemonkey (Ubuntu)
Fix Released
Undecided
Unassigned
Quantal
Fix Released
Undecided
Unassigned

Bug Description

There are some webapps packages that want to be added. I'm helping with the paperwork here, and will be pre-reviewing them (since time is short) before their FFes are actually granted.

None are yet in quantal.

== webapps-applications ==

Holds the actual desktop files and data for the webapps themselves.

FFe: bug 1046840
Packaging: https://code.launchpad.net/~ubuntu-desktop/webapps-applications/ubuntu

Would require MIRs for:
* python-polib

== webapps-greasemonkey ==

Firefox plugin that handles the javascript magic.

FFe: bug 1040313
Packaging: https://code.launchpad.net/~webapps/webapps-greasemonkey/ubuntu

== unity-firefox-extension ==

Firefox plugin that handles Unity integration.

FFe: bug 1040313
Packaging: https://code.launchpad.net/~ubuntu-desktop/unity-firefox-extension/ubuntu

Michael Terry (mterry)
description: updated
Michael Terry (mterry)
affects: ubuntu → gjs (Ubuntu)
Michael Terry (mterry)
description: updated
Revision history for this message
Michael Terry (mterry) wrote :

polib is approved. Has test suite, builds both python3 and python2 versions, uses dh_pythonX, we're in sync, builds clean, no security history, no important bugs.

Changed in polib (Ubuntu):
status: New → Fix Committed
Revision history for this message
Michael Terry (mterry) wrote :

<mdeslaur> mterry: seeing as how mozjs is ripe with known security vulnerabilities because the standalone version is unmaintained, I'd say it has a pretty slim chance of making it in main
<mdeslaur> mterry: probably worth opening a separate MIR for it

Revision history for this message
Michael Terry (mterry) wrote :

So webapps-applications only uses gjs/mozjs for building. We don't likely want mozjs in main. But what about bundling a copy in webapps-applications?

That way we could avoid a loose mojzs in main. Assigning to security team for a check on this crazy idea.

Changed in mozjs (Ubuntu):
assignee: nobody → Ubuntu Security Team (ubuntu-security)
Revision history for this message
Robert Carr (robertcarr) wrote :

It's possible (hopefully, not sure about some of the javascript libraries we are using in the test system they might require mozjs features) to replace gjs/mozjs relatively easily (but not trivially) with Seed/libjavascriptcoregtk. libjavascriptcoregtk seems to be already in main.

Revision history for this message
Michael Terry (mterry) wrote :

Robert, security team doesn't want that either. They are trying to reduce usage of webkit as is.

Another idea is to drop the need for JS. It is apparently used for 2 things right now: the test suite and generating bindings. If we could generate bindings at tarball time and use dep8 for the tests, we wouldn't need JS in the build... I think Robert is looking into that now.

Revision history for this message
Michael Terry (mterry) wrote :

OK, looks like we can get a tarball that doesn't need JS to build (except for tests).

no longer affects: mozjs (Ubuntu)
no longer affects: gjs (Ubuntu)
description: updated
Revision history for this message
Michael Terry (mterry) wrote :

unity-firefox-extension looks mostly OK. With the understanding that Chris Coulson looked at it, I've looked it over and it seems OK.

The one issue was that the test suite wasn't being run, but I fixed that in the packaging. And with that fix, I see one of the tests fail. That's being looked into. So once that is fixed, I would approve.

description: updated
Michael Terry (mterry)
Changed in unity-firefox-extension:
status: New → Incomplete
Revision history for this message
Michael Terry (mterry) wrote :

webapps-applications looks fine, after a new release and fixes to the packaging to make it not need gjs. The test suite turns out to not be runnable by Ubuntu after all, since it needs username/passwords to various web sites to work.

But upstream does run it on their Jenkins setup. So I think that will have to suffice.

Nits:
* desktop files would ideally be namespaced (prefixed) with "webapps-", so something like "webapps-amazon.desktop" or something.
* The two default icons (amazon, ubuntuone-music) needn't be installed in hicolor. You already ship them in the private icon stash. Just point the desktop files at the full path.
* Why are the two default .desktop files launched differently? One uses xdg-open and one uses unity-webapps-runner.

Changed in webapps-applications:
status: New → Fix Committed
Michael Terry (mterry)
description: updated
Revision history for this message
Robert Carr (robertcarr) wrote :

Hi Michael! Some comments on the nits:

The naming is a little tricky! the ubuntuone desktop is named the same one as the unity-webapps generated one so it will be replaced once the webapp is initialized. I am open (and enthused) to changing the naming scheme for the R cycle but am unwilling to touch that code at this point as I am not confident of each place which may depend on the naming scheme.

Default icon point seems good to me...trying to puzzle out if there is any reason to have the icon in hicolor

The ubuntu one music app just launches the browser to the u1 music URL at which point the webapp initializes, creating the unity-webapps generated desktop file (which will use unity-webapps-runner).

In the Amazon case it is using a special option to unity-webapps-runner for selecting the proper amazon locale based on geoclue information. Amazon is a special case.

Revision history for this message
Michael Terry (mterry) wrote :

I'll approve greasemonkey plugin too, based on its basic simplicity and the understanding that Chris Coulson looked at it already. Upstream is very active, builds fine.

It doesn't have a test suite yet, which I don't like. So I filed bug 1053621 to track that.

Changed in webapps-greasemonkey:
status: New → Fix Committed
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Regarding mozjs for the testsuite-- disabling the testsuite in the build is fine. If instructions can be given on how to run the tests, I'll add a QRT test for webapps-applications that can be used to run the tests. This can be pulled into automated tests, used for SRUs and by the security team.

Revision history for this message
Michael Terry (mterry) wrote :

In my packaging branch, test has been fixed. Approved.

Changed in unity-firefox-extension:
status: Incomplete → Fix Committed
Revision history for this message
Matthias Klose (doko) wrote :

Override component to main
polib 1.0.0-2 in quantal: universe/python -> main
python-polib 1.0.0-2 in quantal amd64: universe/python/optional -> main
python-polib 1.0.0-2 in quantal armel: universe/python/optional -> main
python-polib 1.0.0-2 in quantal armhf: universe/python/optional -> main
python-polib 1.0.0-2 in quantal i386: universe/python/optional -> main
python-polib 1.0.0-2 in quantal powerpc: universe/python/optional -> main
python-polib-doc 1.0.0-2 in quantal amd64: universe/doc/optional -> main
python-polib-doc 1.0.0-2 in quantal armel: universe/doc/optional -> main
python-polib-doc 1.0.0-2 in quantal armhf: universe/doc/optional -> main
python-polib-doc 1.0.0-2 in quantal i386: universe/doc/optional -> main
python-polib-doc 1.0.0-2 in quantal powerpc: universe/doc/optional -> main
python3-polib 1.0.0-2 in quantal amd64: universe/python/optional -> main
python3-polib 1.0.0-2 in quantal armel: universe/python/optional -> main
python3-polib 1.0.0-2 in quantal armhf: universe/python/optional -> main
python3-polib 1.0.0-2 in quantal i386: universe/python/optional -> main
python3-polib 1.0.0-2 in quantal powerpc: universe/python/optional -> main
16 publications overridden.

Changed in polib (Ubuntu Quantal):
status: Fix Committed → Fix Released
Michael Terry (mterry)
description: updated
description: updated
Michael Terry (mterry)
description: updated
Michael Terry (mterry)
Changed in unity-firefox-extension (Ubuntu Quantal):
status: New → Fix Committed
Changed in webapps-applications (Ubuntu Quantal):
status: New → Fix Committed
Changed in webapps-greasemonkey (Ubuntu Quantal):
status: New → Fix Committed
no longer affects: webapps-applications
no longer affects: webapps-greasemonkey
no longer affects: unity-firefox-extension
Revision history for this message
Adam Conrad (adconrad) wrote :

Override component to main
unity-firefox-extension 2.3.2-0ubuntu1 in quantal: universe/web -> main
libufe-xidgetter0 2.3.2-0ubuntu1 in quantal amd64: universe/libs/optional -> main
libufe-xidgetter0 2.3.2-0ubuntu1 in quantal armel: universe/libs/optional -> main
libufe-xidgetter0 2.3.2-0ubuntu1 in quantal armhf: universe/libs/optional -> main
libufe-xidgetter0 2.3.2-0ubuntu1 in quantal i386: universe/libs/optional -> main
libufe-xidgetter0 2.3.2-0ubuntu1 in quantal powerpc: universe/libs/optional -> main
xul-ext-unity 2.3.2-0ubuntu1 in quantal amd64: universe/web/optional -> main
xul-ext-unity 2.3.2-0ubuntu1 in quantal armel: universe/web/optional -> main
xul-ext-unity 2.3.2-0ubuntu1 in quantal armhf: universe/web/optional -> main
xul-ext-unity 2.3.2-0ubuntu1 in quantal i386: universe/web/optional -> main
xul-ext-unity 2.3.2-0ubuntu1 in quantal powerpc: universe/web/optional -> main
webapps-applications 2.4.6-0ubuntu2 in quantal: universe/gnome -> main
unity-webapps-common 2.4.6-0ubuntu2 in quantal amd64: universe/gnome/optional -> main
unity-webapps-common 2.4.6-0ubuntu2 in quantal armel: universe/gnome/optional -> main
unity-webapps-common 2.4.6-0ubuntu2 in quantal armhf: universe/gnome/optional -> main
unity-webapps-common 2.4.6-0ubuntu2 in quantal i386: universe/gnome/optional -> main
unity-webapps-common 2.4.6-0ubuntu2 in quantal powerpc: universe/gnome/optional -> main
webapps-greasemonkey 2.3.1-0ubuntu1 in quantal: universe/web -> main
xul-ext-websites-integration 2.3.1-0ubuntu1 in quantal amd64: universe/web/extra -> main
xul-ext-websites-integration 2.3.1-0ubuntu1 in quantal armel: universe/web/extra -> main
xul-ext-websites-integration 2.3.1-0ubuntu1 in quantal armhf: universe/web/extra -> main
xul-ext-websites-integration 2.3.1-0ubuntu1 in quantal i386: universe/web/extra -> main
xul-ext-websites-integration 2.3.1-0ubuntu1 in quantal powerpc: universe/web/extra -> main
Override [y|N]? y
23 publications overridden.

Changed in unity-firefox-extension (Ubuntu Quantal):
status: Fix Committed → Fix Released
Changed in webapps-applications (Ubuntu Quantal):
status: Fix Committed → Fix Released
Changed in webapps-greasemonkey (Ubuntu Quantal):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.