Fix a wrong usage of LabelConstants.EXTERNAL_PACKAGE_NAME. PackageIde…

…ntifier#getPackagePath is used for constructing artifact paths (primarily derived artifacts), not package names, so LabelConstants.EXTERNAL_PATH_PREFIX is the correct one to use there. PiperOrigin-RevId: 346772443
hrfuller · Dec 10, 2020 · 8a0a227 · 8a0a227
1 parent 1f9b8ed
commit 8a0a227
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main/java/com/google/devtools/build/lib/cmdline/PackageIdentifier.java b/src/main/java/com/google/devtools/build/lib/cmdline/PackageIdentifier.java
@@ -185,7 +185,7 @@ public PathFragment getSourceRoot() {
   public PathFragment getPackagePath(boolean siblingRepositoryLayout) {
     return repository.isDefault() || repository.isMain() || siblingRepositoryLayout
         ? pkgName
-        : LabelConstants.EXTERNAL_PACKAGE_NAME
+        : LabelConstants.EXTERNAL_PATH_PREFIX
             .getRelative(repository.strippedName())
             .getRelative(pkgName);
   }