Minify + Shrink resources removes String resources

[AllanWang]

Obviously this is the case, and it can be fixed by keeping the R class, but I was wondering if there is hopefully a better alternative somehow. -keepresources looked promising but it seems to be a dexguard feature rather than a proguard feature.

In comparison, keeping all the R classes adds around 60kb to a < 3mb app, which is somewhat significant.

[rubengees]

I had success with this:

-keepclasseswithmembers class **.R$* {
    public static final int define_*;
}

This only keeps the definitions used by AboutLibraries for autodetection but no other Strings.

[mikepenz]

@rubengees but wouldn't this also remove the strings of the libs themself? So do you add them manually to your project in addition?

[rubengees]

Hmm, no it works out of the box in my case. Maybe proguard intelligently recognizes the usage by the library? I do not use shrinkResources, but only proguard itself though.

[AllanWang]

Shrink resources is the issue, not proguard.
I'm going to try what you have plus
public static final int library_*; and see if that works

[mikepenz]

It can't as those are loaded during the runtime. So this means your definition still keeps the resources.

@AllanWang I do not think there is anything I can do about this, or the lib can do about it.

[rubengees]

Just tried it with shrinkResources enabled on my project, that works.

[AllanWang]

@rubengees is your project open sourced?

[AllanWang]

Also since this keeps the whole R class, I'm not sure if this will be better than the original rules

[rubengees]

Yes, here here are the proguard rules: https://github.com/proxer/ProxerAndroid/blob/master/app/proguard-rules.pro

Also since this keeps the whole R class, I'm not sure if this will be better than the original rules

I am not a proguard expert so this may be true. It did help me avoid the 65k field reference limit though, which popped up due to another lib (I only had the rule to keep entire R from the README before).

@mikepenz

It can't as those are loaded during the runtime.

As far as I know, proguard is able to recognize patterns of Strings when using reflection and keeps those in the resources. If that is the case, I do not know, why the define_ statements does not get keeped...

[rubengees]

@AllanWang

From the proguard documentation:

-keepclasseswithmembers [,modifier,...] class_specification
Specifies classes and class members to be preserved, on the condition that all of the
specified class members are present. For example, you may want to keep all
applications that have a main method, without having to list them explicitly.

I think that means that entire R does not get keeped, but only the fields we use or specify with this rule.

[AllanWang]

@rubengees I think it means that it does, since the only constraint is that a class contains a define_ string and it's preserving the entire class rather than just some members. I'm still waiting for travis to build so I can decompile it later. Plus I think the reason it works with define_ is because the rest of the class is kept, otherwise all your library_ strings will be gone too.

(Not a proguard expert either, so what you're saying is ideally what I want to happen)

[mikepenz]

The only thing which can be maybe done in the future is to split up the core, and move all libraries to a different module. So you can either use the module with all the resources, or you manually add the resources you want

[rubengees]

I just decompiled my app and the R file is really small. I think it works the way we want to (I could be mistaken of course).