Skip to content

Commit e64ce22

Browse files
authored
Merge pull request github#15276 from github/redsun82/cmake-drop-internal-transition
Bazel/CMake: drop confusing `_INTERNAL_TRANSITION` suffix
2 parents c9cf2a8 + b93d108 commit e64ce22

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

misc/bazel/cmake/cmake.bzl

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,11 @@ CmakeInfo = provider(
2222
)
2323

2424
def _cmake_name(label):
25-
return ("%s_%s_%s" % (label.workspace_name, label.package, label.name)).replace("/", "_")
25+
ret = ("%s_%s_%s" % (label.workspace_name, label.package, label.name)).replace("/", "_")
26+
internal_transition_suffix = "_INTERNAL_TRANSITION"
27+
if ret.endswith(internal_transition_suffix):
28+
ret = ret[:-len(internal_transition_suffix)]
29+
return ret
2630

2731
def _cmake_file(file):
2832
if not file.is_source:

0 commit comments

Comments
 (0)