Skip to content
This repository was archived by the owner on Nov 10, 2023. It is now read-only.

Commit b4bdb5e

Browse files
IanChildsfacebook-github-bot
authored andcommitted
Back out "Include Android SDK in D8 desugaring"
Reviewed By: ndmitchell fbshipit-source-id: 589ed14cf0e58df8bf6bd8c7fecb5ec433f22436
1 parent 7b6b03a commit b4bdb5e

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

src/com/facebook/buck/android/D8Step.java

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -177,17 +177,14 @@ private int executeInProcess(IsolatedExecutionContext context) throws IOExceptio
177177
minSdkVersion.ifPresent(builder::setMinApiLevel);
178178
primaryDexClassNamesPath.ifPresent(builder::addMainDexListFiles);
179179

180-
ImmutableSet.Builder<Path> absolutePaths = ImmutableSet.builder();
181-
// Always include the Android SDK on the classpath to support JARs/AARs compiled for Android
182-
absolutePaths.add(androidPlatformTarget.getAndroidJar());
183180
if (classpathFiles != null && !classpathFiles.isEmpty()) {
184-
// classpathFiles is needed only for D8 Java 8 desugar
181+
// classpathFiles is needed only for D8 java 8 desugar
182+
ImmutableSet.Builder<Path> absolutePaths = ImmutableSet.builder();
185183
for (Path classpathFile : classpathFiles) {
186184
absolutePaths.add(filesystem.getPathForRelativeExistingPath(classpathFile));
187185
}
186+
builder.addClasspathFiles(absolutePaths.build());
188187
}
189-
builder.addClasspathFiles(absolutePaths.build());
190-
191188
D8Command d8Command = builder.build();
192189
com.android.tools.r8.D8.run(d8Command);
193190

0 commit comments

Comments
 (0)