Use shorthand display form in Label#debugPrint

This makes the output, which is meant to be friendly to humans, more concise. If users want `//foo` to be `print`ed as `//foo:foo`, they can specify `sep = ""` and also print `:foo`, but stripping the name off is not possible without this change. Closes #24840. PiperOrigin-RevId: 713207601 Change-Id: I474fc4eccb1419304368a5c90a22c13b2197b699
bazelbuild · Jan 8, 2025 · e0fada9 · e0fada9
1 parent 0d02e69
commit e0fada9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/cmdline/Label.java b/src/main/java/com/google/devtools/build/lib/cmdline/Label.java
@@ -689,7 +689,7 @@ public void debugPrint(Printer printer, StarlarkThread thread) {
         // ignore
       }
     }
-    printer.append(getDisplayForm(mainRepoMapping));
+    printer.append(getShorthandDisplayForm(mainRepoMapping));
   }
 
   @Override

diff --git a/src/test/shell/bazel/workspace_test.sh b/src/test/shell/bazel/workspace_test.sh
@@ -723,8 +723,8 @@ EOF
   cd main
   bazel build @foo//:bar \
       >& "$TEST_log" || fail "Expected build to succeed"
-  expect_log "@b//blah:blah"
-  expect_not_log "@a//blah:blah"
+  expect_log "@b//blah"
+  expect_not_log "@a//blah"
 }
 
 function test_workspace_addition_change_aspect() {