On Mon, 2013-11-18 at 18:13 +0530, Vishal Bhoj wrote:
> Hi Jon,
>
> +1 to the idea. Lets use the common sdcard location for all the
> platform i.e /storage/sdcard<0/1>. Please submit a patch to remove the
> reference and I'll ack the same.

Do we want to mandate a location for all platforms? The AOSP example for
platforms without an SD card have the storage
at /storage/emulated/legacy not /storage/sdcard0.

Wouldn't we be better just to remove reference to /sdcard from init.rc
(which was added by us and not present upstream)?

--
Tixy

> Regards,
> Vishal
> On 15 November 2013 23:50, Jon Medhurst (Tixy) <tixy@linaro.org>
> wrote:
>         Linaro's patches to system/core/rootdir/init.rc include
>
>             mkdir /mnt/sdcard 0000 system system
>
>         and
>
>             symlink /mnt/sdcard /sdcard
>
>         is it time to drop these now that AOSP documentation examples
>         have
>         external media under /storage/sdcard0,1 etc and the docs [1]
>         say:
>
>                 When configuring a device-specific init.rc script, the
>                 EXTERNAL_STORAGE environment variable must be defined
>         as the
>                 path to the primary external storage. The /sdcard path
>         must also
>                 resolve to the same location, possibly through a
>         symlink.
>
>         For vexpress [2] I added
>
>                 symlink /storage/sdcard0 /sdcard
>
>         but this is currently being overwritten by the generic
>         init.rc.
>
>         [1] http://source.android.com/devices/tech/storage/config.html
>         [2]
>         http://review.android.git.linaro.org/#/c/4563/2/init.partitions.arm-versatileexpress-usb.rc
>
>         --
>         Tixy
>
>
>         _______________________________________________
>         linaro-android mailing list
>         linaro-android@lists.linaro.org
>         http://lists.linaro.org/mailman/listinfo/linaro-android
>
>