Invalidate external repositories when their names change

Without this, old repositories are still cached across the change to
give a name in the WORKSPACE file, which results in annoying warnings.
This should have been included in
a7fea8a1467a89adde75e67bc99bf0e040f71fe9.

Fixes #1166.

--
Change-Id: I831b842dc07b4ca7925f003b68f4838d083b1166
Reviewed-on: https://bazel-review.googlesource.com/c/3660/
MOS_MIGRATED_REVID=122829553

1 files changed, 6 insertions, 0 deletions

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 d6f143f873..cede852c67 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
@@ -189,6 +189,12 @@ public final class RepositoryDelegatorFunction implements SkyFunction {
     return new Fingerprint()
         .addBytes(RuleSerializer.serializeRule(rule).build().toByteArray())
         .addBytes(ruleSpecificData)
+        // This is to make the fingerprint different after adding names to the generated
+        // WORKSPACE files so they will get re-created, because otherwise there are
+        // annoying warnings for all of them.
+        // TODO(bsilver16384@gmail.com): Remove this once everybody's upgraded to the
+        // new WORKSPACE files.
+        .addInt(1)
         .digestAndReset();
   }