diff options
author | Googler <noreply@google.com> | 2017-03-02 22:22:07 +0000 |
---|---|---|
committer | Yue Gan <yueg@google.com> | 2017-03-03 10:38:01 +0000 |
commit | 8ea4f03c2c71a2cfba93be5e31c7904b37b7476c (patch) | |
tree | 4e64fd073c513ee839573d714b5b42d808d64e92 /src/tools/android/java | |
parent | 9314c72c0f7df99fee65ee84d36014cd7caa67a9 (diff) |
error out when encountering classes missed by incremental dexing
--
PiperOrigin-RevId: 149047031
MOS_MIGRATED_REVID=149047031
Diffstat (limited to 'src/tools/android/java')
-rw-r--r-- | src/tools/android/java/com/google/devtools/build/android/dexer/DexFileMerger.java | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/tools/android/java/com/google/devtools/build/android/dexer/DexFileMerger.java b/src/tools/android/java/com/google/devtools/build/android/dexer/DexFileMerger.java index 5f6f265af2..62cb65ca25 100644 --- a/src/tools/android/java/com/google/devtools/build/android/dexer/DexFileMerger.java +++ b/src/tools/android/java/com/google/devtools/build/android/dexer/DexFileMerger.java @@ -196,6 +196,7 @@ class DexFileMerger { ZipEntryPredicates.suffixes(".class", ".dex"), extraFilter))); Collections.sort(filesToProcess, ZipEntryComparator.LIKE_DX); + int unconverted = 0; for (ZipEntry entry : filesToProcess) { String filename = entry.getName(); try (InputStream content = zip.getInputStream(entry)) { @@ -205,6 +206,8 @@ class DexFileMerger { // a byte buffer before effectively calling Dex(byte[]) anyway. out.add(new Dex(ByteStreams.toByteArray(content))); } else if (filename.endsWith(".class")) { + ++unconverted; + System.err.printf("%s not converted to .dex format yet%n", filename); // TODO(b/34949364): Remove this fallback once Blaze incrementally dexes all Jars out.add(DexFiles.toDex(dexer.toDexFile(ByteStreams.toByteArray(content), filename))); } else { @@ -212,6 +215,12 @@ class DexFileMerger { } } } + // Hitting this error indicates Jar files not covered by incremental dexing (b/34949364). This + // is a backstop until we can build with --experimental_incremental_dexing_error_on_missed_jars, + // which prevents this error from happening. If you do get this exception, find the Jar files + // the reported classes come from and depend on them via java_imports. + checkState(unconverted == 0, "Had to convert %s files in %s on the fly", unconverted, + zip.getName()); } private static DexFileAggregator createDexFileAggregator(Options options) throws IOException { |