From 34771a163829d87ec0bcb175c8ec78a8a988d1b3 Mon Sep 17 00:00:00 2001 From: David Ostrovsky Date: Sat, 25 Jul 2020 17:45:46 +0200 Subject: [PATCH] Allow hyphen char in workspace name Closes #11837. --- .../google/devtools/build/lib/packages/WorkspaceGlobals.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/google/devtools/build/lib/packages/WorkspaceGlobals.java b/src/main/java/com/google/devtools/build/lib/packages/WorkspaceGlobals.java index 719873637f382f..ab98b38f460358 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/WorkspaceGlobals.java +++ b/src/main/java/com/google/devtools/build/lib/packages/WorkspaceGlobals.java @@ -49,8 +49,8 @@ /** A collection of global Starlark build API functions that apply to WORKSPACE files. */ public class WorkspaceGlobals implements WorkspaceGlobalsApi { - // Must start with a letter and can contain letters, numbers, and underscores - private static final Pattern LEGAL_WORKSPACE_NAME = Pattern.compile("^\\p{Alpha}\\w*$"); + // Must start with a letter and can contain letters, numbers, underscores and hyphens. + private static final Pattern LEGAL_WORKSPACE_NAME = Pattern.compile("^\\p{Alpha}[\\w-]*$"); private final boolean allowOverride; private final RuleFactory ruleFactory;