commit 6da79e94b0af7a82cffe8f65bc53911fbc44b5b6
author Adam Cozzette <acozzette@google.com> Wed Aug 19 11:32:03 2020 -0700
committer Adam Cozzette <acozzette@google.com> Fri Aug 21 16:33:52 2020 -0700
tree 10d0cf24d3ef2d5418718c67d67f9153841ae8b5
parent 4b93993ac714342e57aa3c47dfc07c9fb0e4c181

Fixed VERSION_SUFFIX logic in update_version.py

This commit makes a couple of fixes:
- Make sure we always update the suffix, since even for a non-RC release
  we need to clear the suffix.
- Make sure the suffix is properly quoted and begins with -rc

update_version.py

diff --git a/update_version.py b/update_version.py
index 10410e0..bfbef45 100755
--- a/update_version.py
+++ b/update_version.py
@@ -101,6 +101,9 @@
 def UpdateCpp():
   cpp_version = '%d%03d%03d' % (
     NEW_VERSION_INFO[0], NEW_VERSION_INFO[1], NEW_VERSION_INFO[2])
+  version_suffix = ''
+  if RC_VERSION != -1:
+    version_suffix = '-rc%s' % RC_VERSION
   def RewriteCommon(line):
     line = re.sub(
       r'^#define GOOGLE_PROTOBUF_VERSION .*$',
@@ -110,15 +113,14 @@
       r'^#define PROTOBUF_VERSION .*$',
       '#define PROTOBUF_VERSION %s' % cpp_version,
       line)
-    if RC_VERSION != -1:
-      line = re.sub(
-          r'^#define GOOGLE_PROTOBUF_VERSION_SUFFIX .*$',
-          '#define GOOGLE_PROTOBUF_VERSION_SUFFIX %s' % RC_VERSION,
-          line)
-      line = re.sub(
-          r'^#define PROTOBUF_VERSION_SUFFIX .*$',
-          '#define PROTOBUF_VERSION_SUFFIX %s' % RC_VERSION,
-          line)
+    line = re.sub(
+        r'^#define GOOGLE_PROTOBUF_VERSION_SUFFIX .*$',
+        '#define GOOGLE_PROTOBUF_VERSION_SUFFIX "%s"' % version_suffix,
+        line)
+    line = re.sub(
+        r'^#define PROTOBUF_VERSION_SUFFIX .*$',
+        '#define PROTOBUF_VERSION_SUFFIX "%s"' % version_suffix,
+        line)
     if NEW_VERSION_INFO[2] == 0:
       line = re.sub(
         r'^#define PROTOBUF_MIN_HEADER_VERSION_FOR_PROTOC .*$',
@@ -137,17 +139,16 @@
         'static const int kMinHeaderVersionForProtoc = %s;' % cpp_version,
         line)
     return line
-  
+
   def RewritePortDef(line):
     line = re.sub(
       r'^#define PROTOBUF_VERSION .*$',
       '#define PROTOBUF_VERSION %s' % cpp_version,
       line)
-    if RC_VERSION != -1:
-      line = re.sub(
-          r'^#define PROTOBUF_VERSION_SUFFIX .*$',
-          '#define PROTOBUF_VERSION_SUFFIX %s' % RC_VERSION,
-          line)
+    line = re.sub(
+        r'^#define PROTOBUF_VERSION_SUFFIX .*$',
+        '#define PROTOBUF_VERSION_SUFFIX "%s"' % version_suffix,
+        line)
     if NEW_VERSION_INFO[2] == 0:
       line = re.sub(
         r'^#define PROTOBUF_MIN_HEADER_VERSION_FOR_PROTOC .*$',