desword / android-apktool

Automatically exported from code.google.com/p/android-apktool
Other
1 stars 0 forks source link

Errors when used to rebuild a Corona SDK's APK file #657

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Disassemble a Corona SDK apk file
2. Rebuild it

What is the expected output? What do you see instead?
A correct APK should be built.

What version of the product are you using? On what operating system?
Apktool v1.5.2, Mac OS X 10.9.2

There are two problems inside AndroidManifest.xml
1. For some reason package name is wrong and set to "com.ansca.corona".
2. If Google Play Game Services are used, APP_ID is treated as number, but 
should be a string. The solution is to prepend the ID with an escaped space 
character.

<meta-data android:name="com.google.android.gms.games.APP_ID" android:value="\ 
555555555555"/>

After that everything is fine.

Original issue reported on code.google.com by lvs.tour...@gmail.com on 15 Jul 2014 at 11:32

GoogleCodeExporter commented 9 years ago
hmm.

https://developers.google.com/games/services/android/quickstart

Looking here. It says you should use a @string reference

   <meta-data android:name="com.google.android.gms.games.APP_ID" android:value="@string/app_id" />
   <meta-data android:name="com.google.android.gms.version" android:value="@integer/google_play_services_version"/>

Which I then believe is the same bug as #526 (broken references in 
AndroidManifest). Either way, I haven't built my test apk to investigate fully 
yet. So will leave open.

Original comment by connor.tumbleson on 20 Jul 2014 at 2:53

GoogleCodeExporter commented 9 years ago
1) was already fixed in 2.x
2) Yes duplicated. Strings that "look" like ints are casted to ints, thus 
exceed MAX_INT and get some weird stuff going on.

Original comment by connor.tumbleson on 20 Jul 2014 at 3:06

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Issue 780 has been merged into this issue.

Original comment by connor.tumbleson on 13 Mar 2015 at 2:41