Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3762)

Issue 3805047: Allow for SD Card Installation

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 3 months ago by GJTorikian
Modified:
9 years, 4 months ago
CC:
npr-android-app_googlegroups.com
Base URL:
http://npr-android-app.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Add new attribute to AndroidManifest.xml: android:installLocation="auto" Default is phone internal storage, though user can also move to SD card Change project's build from 2.1-update1 to 2.2. Shouldn't have any impact on the app as it's not using deprecated APIs. EDITED: I didn't realize I had to actually select the reviewer. I hope adding Michael is all right.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
AndroidManifest.xml View 1 chunk +1 line, -1 line 0 comments Download
default.properties View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2
GJTorikian
13 years, 3 months ago (2011-01-08 19:20:08 UTC) #1
jeremy.wadsack
13 years, 2 months ago (2011-02-28 18:02:32 UTC) #2
As Michael mentioned on the mailing list when this was posted (Issue 40 in the
tracker), applications with Services are recommended to NOT be installed on SD
cards.

http://developer.android.com/guide/appendix/install-location.html#ShouldNot

---------
If we do that, we have to workaround the following issue:

Services
Your running Service will be killed and will not be restarted when external
storage is remounted. You can, however, register for the
ACTION_EXTERNAL_APPLICATIONS_AVAILABLE
---------

So basically we need to add a broadcast receiver to the application for
http://developer.android.com/reference/android/content/Intent.html#ACTION_EXT....
Also, the Service/Application needs to be know that it was playing when it was
killed to it can restart on receipt of this notice.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b