commit 5b05b8a78d72df45dce6bb5b58c0b214ab72b6ef
author Googler <noreply@google.com> Mon Nov 05 23:54:03 2018 -0800
committer Copybara-Service <copybara-piper@google.com> Mon Nov 05 23:55:58 2018 -0800
tree 51222b7ec0d7b70610d72b4d0f8c02cbf7e8c7a7
parent 4de671d7b27412503d9b242c1062214b80a9fb3e

Remove extra '.' in error msg.

RELNOTES: None
PiperOrigin-RevId: 220234096

src/main/java/com/google/devtools/build/lib/syntax/SkylarkImports.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkImports.java b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkImports.java
index a339b7c..9857f0f 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkImports.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkImports.java
@@ -154,7 +154,7 @@
 
   @VisibleForTesting
   static final String INVALID_PATH_SYNTAX =
-      "First argument of 'load' must be a label and start with either '//', ':', or '@'.";
+      "First argument of 'load' must be a label and start with either '//', ':', or '@'";
 
   @VisibleForTesting
   static final String INVALID_TARGET_PREFIX = "Invalid target: ";

src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java b/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
index 682a8a4..438c9f4 100644
--- a/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/syntax/SkylarkEvaluationTest.java
@@ -2224,35 +2224,35 @@
   @Test
   public void testLoadStatementWithAbsolutePath() throws Exception {
     checkEvalErrorContains(
-        "First argument of 'load' must be a label and start with either '//', ':', or '@'.",
+        "First argument of 'load' must be a label and start with either '//', ':', or '@'",
         "load('/tmp/foo', 'arg')");
   }
 
   @Test
   public void testLoadStatementWithRelativePath() throws Exception {
     checkEvalErrorContains(
-        "First argument of 'load' must be a label and start with either '//', ':', or '@'.",
+        "First argument of 'load' must be a label and start with either '//', ':', or '@'",
         "load('foo', 'arg')");
   }
 
   @Test
   public void testLoadStatementWithExternalLabel() throws Exception {
     checkEvalErrorDoesNotContain(
-        "First argument of 'load' must be a label and start with either '//', ':', or '@'.",
+        "First argument of 'load' must be a label and start with either '//', ':', or '@'",
         "load('@other//foo.bzl', 'arg')");
   }
 
   @Test
   public void testLoadStatementWithAbsoluteLabel() throws Exception {
     checkEvalErrorDoesNotContain(
-        "First argument of 'load' must be a label and start with either '//', ':', or '@'.",
+        "First argument of 'load' must be a label and start with either '//', ':', or '@'",
         "load('//foo.bzl', 'arg')");
   }
 
   @Test
   public void testLoadStatementWithRelativeLabel() throws Exception {
     checkEvalErrorDoesNotContain(
-        "First argument of 'load' must be a label and start with either '//', ':', or '@'.",
+        "First argument of 'load' must be a label and start with either '//', ':', or '@'",
         "load(':foo.bzl', 'arg')");
   }