2019-07-25 11:18:50 +00:00
|
|
|
diff --git a/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryDelegatorFunction.java b/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryDelegatorFunction.java
|
|
|
|
index 8e772005cd..6ffa1c919c 100644
|
|
|
|
--- a/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryDelegatorFunction.java
|
|
|
|
+++ b/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryDelegatorFunction.java
|
|
|
|
@@ -432,25 +432,7 @@ public final class RepositoryDelegatorFunction implements SkyFunction {
|
|
|
|
String content;
|
|
|
|
try {
|
|
|
|
content = FileSystemUtils.readContent(markerPath, StandardCharsets.UTF_8);
|
|
|
|
- String markerRuleKey = readMarkerFile(content, markerData);
|
|
|
|
- boolean verified = false;
|
|
|
|
- if (Preconditions.checkNotNull(ruleKey).equals(markerRuleKey)
|
|
|
|
- && Objects.equals(
|
|
|
|
- markerData.get(MANAGED_DIRECTORIES_MARKER),
|
|
|
|
- this.markerData.get(MANAGED_DIRECTORIES_MARKER))) {
|
|
|
|
- verified = handler.verifyMarkerData(rule, markerData, env);
|
|
|
|
- if (env.valuesMissing()) {
|
|
|
|
- return null;
|
|
|
|
- }
|
|
|
|
- }
|
|
|
|
-
|
|
|
|
- if (verified) {
|
|
|
|
- return new Fingerprint().addString(content).digestAndReset();
|
|
|
|
- } else {
|
|
|
|
- // So that we are in a consistent state if something happens while fetching the repository
|
|
|
|
- markerPath.delete();
|
|
|
|
- return null;
|
|
|
|
- }
|
|
|
|
+ return new Fingerprint().addString(content).digestAndReset();
|
|
|
|
} catch (IOException e) {
|
|
|
|
throw new RepositoryFunctionException(e, Transience.TRANSIENT);
|
|
|
|
}
|
|
|
|
diff --git a/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java b/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java
|
|
|
|
index c282d57ab6..f9b0c08627 100644
|
|
|
|
--- a/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java
|
|
|
|
+++ b/src/main/java/com/google/devtools/build/lib/shell/JavaSubprocessFactory.java
|
|
|
|
@@ -146,7 +146,6 @@ public class JavaSubprocessFactory implements SubprocessFactory {
|
2017-10-15 11:45:19 +00:00
|
|
|
ProcessBuilder builder = new ProcessBuilder();
|
|
|
|
builder.command(params.getArgv());
|
|
|
|
if (params.getEnv() != null) {
|
|
|
|
- builder.environment().clear();
|
|
|
|
builder.environment().putAll(params.getEnv());
|
|
|
|
}
|
|
|
|
|