Last Comment Bug 676723 - change unofficial android package name based on user
: change unofficial android package name based on user
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Widget: Android (show other bugs)
: Trunk
: ARM Android
: -- normal (vote)
: mozilla8
Assigned To: Brad Lassey [:blassey] (use needinfo?)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-04 16:38 PDT by Brad Lassey [:blassey] (use needinfo?)
Modified: 2011-08-10 08:33 PDT (History)
11 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (1.30 KB, patch)
2011-08-04 16:51 PDT, Brad Lassey [:blassey] (use needinfo?)
mbrubeck: review+
Details | Diff | Review

Description Brad Lassey [:blassey] (use needinfo?) 2011-08-04 16:38:40 PDT
Its sometimes a pain to install another developer's build because you need to uninstall your own and hence blow away your profile.

I'm proposing that we change the package name of unofficial builds from org.mozilla.fennec_unofficial to org.mozilla.fennec_blassey (in my case obviously).

Anyone see a drawback for this?
Comment 1 Brad Lassey [:blassey] (use needinfo?) 2011-08-04 16:51:48 PDT
Created attachment 550892 [details] [diff] [review]
patch
Comment 2 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2011-08-04 16:52:56 PDT
$USER can be an arbitrary string, but other than that, sounds OK.  I have scripts that reference the .apk but they're already using "fennec-*.apk" as I assume other folks do.  There's a small information leak by including the username in the package name, but probably nothing worse than you can find in about:buildconfig.
Comment 3 Matt Brubeck (:mbrubeck) 2011-08-04 16:59:31 PDT
Comment on attachment 550892 [details] [diff] [review]
patch

Looks good to me.  If we do this, let's push this at the same time as the new patch in bug 669424, to minimize the number of times developers will need to wipe and reinstall their local builds.

The upside of this change is that I can install test builds from other developers without affecting my own local build.  The flip side is that I'll have to start from a fresh profile for each developer's build I test; I won't already have sync or other preferences set up.  Also, when installing multiple development builds they will all have the same name and icon.  I can live with those issues, since we don't need to ship development builds around very often.
Comment 4 Mark Finkle (:mfinkle) (use needinfo?) 2011-08-04 17:19:56 PDT
This also affects testing local builds too. Where before we could always assume 1 of 4 different package names, now we'll have to worry about the user who built the app.
Comment 5 Brad Lassey [:blassey] (use needinfo?) 2011-08-04 19:12:51 PDT
(In reply to comment #2)
> $USER can be an arbitrary string, but other than that, sounds OK.  I have
> scripts that reference the .apk but they're already using "fennec-*.apk" as
> I assume other folks do.  There's a small information leak by including the
> username in the package name, but probably nothing worse than you can find
> in about:buildconfig.

that's a different package name. The apk file name will stay the same (fennec-8.0a1.en-US.android-arm.apk), its the java package name that changes and with it the android application id.
Comment 6 Doug Turner (:dougt) 2011-08-05 09:38:56 PDT
much easier to upload my builds to the store!
Comment 7 Brad Lassey [:blassey] (use needinfo?) 2011-08-09 22:12:41 PDT
http://hg.mozilla.org/integration/mozilla-inbound/rev/385b8dfae4c0
Comment 8 :Ehsan Akhgari (out sick) 2011-08-10 08:33:11 PDT
http://hg.mozilla.org/mozilla-central/rev/385b8dfae4c0

Note You need to log in before you can comment on or make changes to this bug.