aboutsummaryrefslogtreecommitdiffstats
path: root/bin/post_version.py
diff options
context:
space:
mode:
authorDylan Baker <[email protected]>2019-10-09 10:31:16 -0700
committerDylan Baker <[email protected]>2019-10-25 15:46:06 -0700
commitabf9e7ac7bdb26ef203394ccdd5f0a43ceda1493 (patch)
tree601545d2962dff630af73b4b9030a04265c66dd5 /bin/post_version.py
parentc6d41e7f0b0c998d9b6f4b1ff55eab9f6cd12ef4 (diff)
bin/post_version.py: Pass version as an argument
I made a bad assumption; I assumed this would be run in the release branch. But we don't do that, we run in the master branch. As a result we need to pass the version as an argument. Fixes: 3226b12a09bbcbd25526fd6da6257057d26ddb31 ("release: Add an update_release_calendar.py script") Reviewed-by: Eric Engestrom <[email protected]> Reviewed-by: Juan A. Suarez <[email protected]>
Diffstat (limited to 'bin/post_version.py')
-rwxr-xr-xbin/post_version.py26
1 files changed, 10 insertions, 16 deletions
diff --git a/bin/post_version.py b/bin/post_version.py
index 9afb37b18a3..2d195dce1b3 100755
--- a/bin/post_version.py
+++ b/bin/post_version.py
@@ -21,6 +21,7 @@
"""Update the main page, release notes, and calendar."""
+import argparse
import calendar
import datetime
import pathlib
@@ -51,18 +52,8 @@ def calculate_previous_version(version: str, is_point: bool) -> str:
return '.'.join(base)
-def get_version() -> str:
- v = pathlib.Path(__file__).parent.parent / 'VERSION'
- with v.open('rt') as f:
- raw_version = f.read().strip()
- return raw_version.split('-')[0]
-
-
-def is_point_release() -> bool:
- v = pathlib.Path(__file__).parent.parent / 'VERSION'
- with v.open('rt') as f:
- raw_version = f.read().strip()
- return '-rc' not in raw_version
+def is_point_release(version: str) -> bool:
+ return not version.endswith('.0')
def update_index(is_point: bool, version: str, previous_version: str) -> None:
@@ -110,11 +101,14 @@ def update_release_notes(previous_version: str) -> None:
def main() -> None:
- is_point = is_point_release()
- version = get_version()
- previous_version = calculate_previous_version(version, is_point)
+ parser = argparse.ArgumentParser()
+ parser.add_argument('version', help="The released version.")
+ args = parser.parse_args()
+
+ is_point = is_point_release(args.version)
+ previous_version = calculate_previous_version(args.version, is_point)
- update_index(is_point, version, previous_version)
+ update_index(is_point, args.version, previous_version)
update_release_notes(previous_version)