Skip to content

Commit 25efb74

Browse files
committed
[Bazel] Use PACKAGE_VERSION for version string.
This enables "-rc" suffix in release branches.
1 parent eee620b commit 25efb74

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

utils/bazel/llvm-project-overlay/clang/BUILD.bazel

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@
44

55
load(
66
"//:vars.bzl",
7-
"LLVM_VERSION",
87
"LLVM_VERSION_MAJOR",
98
"LLVM_VERSION_MINOR",
109
"LLVM_VERSION_PATCH",
10+
"PACKAGE_VERSION",
1111
)
1212
load("//:workspace_root.bzl", "workspace_root")
1313
load("//llvm:binary_alias.bzl", "binary_alias")
@@ -553,12 +553,12 @@ genrule(
553553
"echo '#define CLANG_VERSION_MAJOR_STRING \"{major}\"' >> $@\n" +
554554
"echo '#define CLANG_VERSION_MINOR {minor}' >> $@\n" +
555555
"echo '#define CLANG_VERSION_PATCHLEVEL {patch}' >> $@\n" +
556-
"echo '#define CLANG_VERSION_STRING \"{vers}git\"' >> $@\n"
556+
"echo '#define CLANG_VERSION_STRING \"{vers}\"' >> $@\n"
557557
).format(
558558
major = LLVM_VERSION_MAJOR,
559559
minor = LLVM_VERSION_MINOR,
560560
patch = LLVM_VERSION_PATCH,
561-
vers = LLVM_VERSION,
561+
vers = PACKAGE_VERSION,
562562
),
563563
)
564564

utils/bazel/llvm-project-overlay/llvm/config.bzl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66

77
load(
88
"//:vars.bzl",
9-
"LLVM_VERSION",
109
"LLVM_VERSION_MAJOR",
1110
"LLVM_VERSION_MINOR",
1211
"LLVM_VERSION_PATCH",
12+
"PACKAGE_VERSION",
1313
)
1414

1515
def native_arch_defines(arch, triple):
@@ -108,7 +108,7 @@ llvm_config_defines = os_defines + builtin_thread_pointer + select({
108108
"LLVM_VERSION_MAJOR={}".format(LLVM_VERSION_MAJOR),
109109
"LLVM_VERSION_MINOR={}".format(LLVM_VERSION_MINOR),
110110
"LLVM_VERSION_PATCH={}".format(LLVM_VERSION_PATCH),
111-
r'LLVM_VERSION_STRING=\"{}git\"'.format(LLVM_VERSION),
111+
r'LLVM_VERSION_STRING=\"{}\"'.format(PACKAGE_VERSION),
112112
# These shouldn't be needed by the C++11 standard, but are for some
113113
# platforms (e.g. glibc < 2.18. See
114114
# https://sourceware.org/bugzilla/show_bug.cgi?id=15366). These are also

0 commit comments

Comments
 (0)