Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIMOB-24363] Prevent duplicate entries in APK #9338

Merged
merged 2 commits into from
Aug 29, 2017

Conversation

garymathews
Copy link
Contributor

  • Do not include duplicate entries into the unsigned APK
  • Prevents jarsigner: unable to sign jar: java.util.zip.ZipException: duplicate entry error

JIRA Ticket

@@ -4149,6 +4150,12 @@ AndroidBuilder.prototype.createUnsignedApk = function createUnsignedApk(next) {
&& !classRegExp.test(entry.name)
&& !trailingSlashRegExp.test(entry.entryName)
) {
// do not add duplicate entries
if (entryNames.indexOf(entry.entryName) > -1) {
this.logger.warn(__('Removing duplicate entry %s', entry.entryName.cyan));
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cb1kenobi warn does not print any output (even with -l trace), switching to info does. Any ideas?

@lokeshchdhry
Copy link
Contributor

FR Passed.

No duplicate entry: assets/README error is seen.

Studio Ver: 4.9.1.201707200100
SDK Ver: 7.0.0 local build
OS Ver: 10.12.3
Xcode Ver: Xcode 8.3.3
Appc NPM: 4.2.9
Appc CLI: 6.2.3
Ti CLI Ver: 5.0.14
Alloy Ver: 1.9.13
Node Ver: 6.10.1
Java Ver: 1.8.0_101
Devices: ⇨ google Nexus 5 --- Android 6.0.1
⇨ google Pixel --- Android 7.1.1

@lokeshchdhry lokeshchdhry merged commit 6726165 into tidev:master Aug 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants