Closed Bug 1063546 Opened 10 years ago Closed 8 years ago

Engineering apps should be uninstallable on engineering builds

Categories

(Firefox OS Graveyard :: Gaia::Build, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: martijn.martijn, Unassigned)

Details

I chatted with Fernando about this. He said engineering apps should be uninstallable.
On trunk, it's currently not.

I found this out, while trying to uninstall the uitest-privileged app.
This is the part of the webapps.json file for this app on trunk:
      "uitest-privileged.gaiamobile.org": {
        "origin": "app://uitest-privileged.gaiamobile.org",
        "installOrigin": "app://uitest-privileged.gaiamobile.org",
        "receipt": null,
        "installTime": 1409925910312,
        "updateTime": 1409925910312,
        "manifestURL": "app://uitest-privileged.gaiamobile.org/manifest.webapp",
        "localId": 1056,
        "appStatus": 2,
        "basePath": "/system/b2g/webapps",
        "id": "uitest-privileged.gaiamobile.org",
        "removable": false,
        "name": "UI tests - Privileged App",
        "csp": "",
        "role": "",
        "widgetPages": [],
        "redirects": null,
        "kind": "packaged"
      },
Flags: needinfo?(yurenju.mozilla)
Martijn, I run gaia by b2g-desktop but uitest-privileged is removable since property "removable" is true, so I believe it depends on platform.

we can make all apps can be removable in build process[1] for engineering build but I would like to know the reason for that.

[1] https://github.com/mozilla-b2g/gaia/blob/master/build/copy-build-stage-data.js#L64-L76
Flags: needinfo?(yurenju.mozilla) → needinfo?(martijn.martijn)
(In reply to Yuren [:yurenju] from comment #1)
> we can make all apps can be removable in build process[1] for engineering
> build but I would like to know the reason for that.

That is what Fernando told me, that it should be the case. I can't really give any other reason, apart from the reason that it can be really handy to be able to uninstall (and reinstall) some of these apps.
Flags: needinfo?(martijn.martijn) → needinfo?(ferjmoreno)
(In reply to Yuren [:yurenju] from comment #1)
> Martijn, I run gaia by b2g-desktop but uitest-privileged is removable since
> property "removable" is true, so I believe it depends on platform.
> 
> we can make all apps can be removable in build process[1] for engineering
> build but I would like to know the reason for that.
> 
> [1]
> https://github.com/mozilla-b2g/gaia/blob/master/build/copy-build-stage-data.
> js#L64-L76

In this case the reason is to ease the process of testing some features that has application data associated that can only be deleted via app removal (or resetting the phone, which is obviously not desired).
Flags: needinfo?(ferjmoreno)
I suspect if this bug would be not there, we would have caught bug 1075070 earlier.
Marking WONFTIX, sorry for the bug spam. If somebody still wants to work on this, please open a new bug.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.