OSDN Git Service

AAPT: treat "-I" with lower precedence than main APK for dumping.
authorDonald Chai <dchai@google.com>
Fri, 13 Oct 2017 04:00:45 +0000 (21:00 -0700)
committerDonald Chai <dchai@google.com>
Fri, 13 Oct 2017 04:34:22 +0000 (21:34 -0700)
commitd1ac6e1f9ea3a4d6c0d6d4a118c61526b8234a5f
tree8e9697838fb471962de098194c91519e71863069
parent96a30ab84c9e856cd7152d66d603205c3daff36e
AAPT: treat "-I" with lower precedence than main APK for dumping.

This is the desired behavior if resource IDs may overlap, and also happens to
improve the behavior of:

  aapt dump -I base.apk badging feature.apk

when both APKs use the same package name with different package IDs
(base=0x7F, feature=0x80).  Previously, the final call to
DynamicRefTable::addMapping was for base.apk, and all references to 0x80 were
rewritten in DynamicRefTable::lookupResourceId to 0x7F.  After this
change resources defined in both APKs are resolved correctly.

Note that this is a bit of kludge, and APKs should use different names
to avoid conflating the package IDs.

Fixes: 67070085
Bug: 64058531
Test: manual with sample in b/67070085#comment2
Change-Id: I285adb6f44a297440b08fc7a1f9ad73c700eb9bc
tools/aapt/Command.cpp