Prepare git_update_version.py for real usage.

Message ID 2207388c-0d0b-578b-029b-b77128885cbf@suse.cz
State New
Headers show
Series
  • Prepare git_update_version.py for real usage.
Related show

Commit Message

Martin Liška May 21, 2020, 9:51 a.m.
Hi.

I've just installed changed that are prepared for script usage
at server.

Martin

contrib/ChangeLog:

2020-05-21  Martin Liska  <mliska@suse.cz>

	* gcc-changelog/git_update_version.py: Prepare the script, the
	only missing piece is pushing of the updated branches.
---
  contrib/gcc-changelog/git_update_version.py | 17 ++++++++---------
  1 file changed, 8 insertions(+), 9 deletions(-)

Patch

diff --git a/contrib/gcc-changelog/git_update_version.py b/contrib/gcc-changelog/git_update_version.py
index f38e011df45..ee3989beb27 100755
--- a/contrib/gcc-changelog/git_update_version.py
+++ b/contrib/gcc-changelog/git_update_version.py
@@ -24,8 +24,7 @@  from git import Repo
 
 from git_repository import parse_git_revisions
 
-# TODO: remove sparta suffix
-current_timestamp = datetime.datetime.now().strftime('%Y%m%d sparta\n')
+current_timestamp = datetime.datetime.now().strftime('%Y%m%d\n')
 
 
 def read_timestamp(path):
@@ -36,11 +35,9 @@  def prepend_to_changelog_files(repo, folder, git_commit):
     if not git_commit.success:
         for error in git_commit.errors:
             print(error)
-        # TODO: add error message
-        return
+        raise AssertionError()
     for entry, output in git_commit.to_changelog_entries(use_commit_ts=True):
-        # TODO
-        full_path = os.path.join(folder, entry, 'ChangeLog.test')
+        full_path = os.path.join(folder, entry, 'ChangeLog')
         print('writting to %s' % full_path)
         if os.path.exists(full_path):
             content = open(full_path).read()
@@ -73,9 +70,10 @@  for ref in origin.refs:
             branch = repo.branches[name]
         else:
             branch = repo.create_head(name, ref).set_tracking_branch(ref)
+        print('=== Working on: %s ===' % branch, flush=True)
         origin.pull(rebase=True)
         branch.checkout()
-        print('=== Working on: %s ===' % branch)
+        print('branch pulled and checked out')
         assert not repo.index.diff(None)
         commit = branch.commit
         commit_count = 1
@@ -90,9 +88,8 @@  for ref in origin.refs:
         datestamp_path = os.path.join(args.git_path, 'gcc/DATESTAMP')
         if read_timestamp(datestamp_path) != current_timestamp:
             print('DATESTAMP will be changed:')
-            # TODO: set strict=True after testing period
             commits = parse_git_revisions(args.git_path, '%s..HEAD'
-                                          % commit.hexsha, strict=False)
+                                          % commit.hexsha)
             for git_commit in reversed(commits):
                 prepend_to_changelog_files(repo, args.git_path, git_commit)
             # update timestamp
@@ -101,5 +98,7 @@  for ref in origin.refs:
             repo.git.add(datestamp_path)
             repo.index.commit('Daily bump.')
             # TODO: push the repository
+            # repo.git.push('origin', branch)
         else:
             print('DATESTAMP unchanged')
+        print('branch is done\n', flush=True)