[Android] Bug #10691 Proposed solution

Felix Abecassis felix.abecassis at gmail.com
Thu Jul 17 18:11:14 CEST 2014


2014-07-17 18:03 GMT+02:00 Edward Wang <edward.c.wang at compdigitec.com>:
> On Thu, Jul 17, 2014 at 11:52 AM, Felix Abecassis
> <felix.abecassis at gmail.com> wrote:
>> 2014-07-17 17:32 GMT+02:00 Rocky Backer <rocky.backer at gmail.com>:
>>> Hi,
>>>
>>> Here is a solution for bug #10691 titled "Use different icon for nightly
>>> builds of VLC for Android".
>>> Link: https://trac.videolan.org/vlc/ticket/10691
>>>
>>> 1. Maintain two copies of icon assets, orange and blue.
>>> 2. Have a flag NIGHTLY just like RELEASE, to be set to 1 if it is a nightly
>>> build.
>>> 3. In the Makefile, read NIGHTLY flag and if set, copy the blue icon assets
>>> into the drawable folders before building the apk.
>>>
>>> Is this solution alright?
>>>
>>
>> I think the RELEASE flag is set when using ./compile.sh release, with
>> your solution we would need to add a new command line argument to
>> compile.sh, no? Or we would need to set the env variable manually in
>> our shell.
>> Maybe VLC-debug.apk should have the blue icon, and VLC-release-*.apk
>> should have the orange icon, but I'm not sure that's simple (or
>> possible).
>
> Or whether we even need this at all - I mean VLC nightly builds on
> Windows don't have a custom icon either and users seem to be getting
> along fine.
>
> You can tell if a nightly build is being used by looking at the
> version - it will have a "-git" postfixed to it.
>

Sure, but when you have the store version and a custom build installed
on your device, it's hard to tell which is which: you have two
applications with the same name and the same icon. With a different
icon you would be able to tell at a glance. But yes, it's a minor
feature.

> Regards,
> Edward
> _______________________________________________
> Android mailing list
> Android at videolan.org
> https://mailman.videolan.org/listinfo/android



-- 
Félix Abecassis
http://felix.abecassis.me


More information about the Android mailing list