As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact bugzilla-admin@mozilla.org
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?)
:
: Jim Chen [:jchen] [:darchons]
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 | Splinter Review

Description User image 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 User image Brad Lassey [:blassey] (use needinfo?) 2011-08-04 16:51:48 PDT
Created attachment 550892 [details] [diff] [review]
patch
Comment 2 User image 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 User image 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 User image 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 User image 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 User image Doug Turner (:dougt) 2011-08-05 09:38:56 PDT
much easier to upload my builds to the store!
Comment 7 User image Brad Lassey [:blassey] (use needinfo?) 2011-08-09 22:12:41 PDT
http://hg.mozilla.org/integration/mozilla-inbound/rev/385b8dfae4c0

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