jenkins-bot has submitted this change and it was merged.
Change subject: Returns unknown version when git repo is cleaned, instead of crashing
......................................................................
Returns unknown version when git repo is cleaned, instead of crashing
When the git repo is clean, version.py can't open the git version file
to get hash and crashes, causes crash of all scripts.
Bug: T91904
Change-Id: Iaa6458e60e42bf3026ff56af9e797ec580e6bb85
---
M pywikibot/version.py
1 file changed, 8 insertions(+), 3 deletions(-)
Approvals:
John Vandenberg: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/version.py b/pywikibot/version.py
index 3eba2a0..ee49fe7 100644
--- a/pywikibot/version.py
+++ b/pywikibot/version.py
@@ -117,7 +117,7 @@
date = time.strptime(date, "%Y-%m-%d %H:%M:%S")
return rev, date
if not rev:
- return "(unknown)", time.strptime("2000-01-01 00:00:00",
"%Y-%m-%d %H:%M:%S")
+ raise ParseError
return rev, date
@@ -185,8 +185,13 @@
subprocess.Popen([cmd], stdout=subprocess.PIPE).communicate()
except:
#Means git hasn't been installed, no way to get the date, retreving date
from gitblit
- hsh = open(os.path.join(_program_dir, '.git/refs/heads/master'),
'r').read().strip(" \n")
- rev, date = getversion_git_windows(hsh, path)
+ file_path = os.path.join(_program_dir, '.git/refs/heads/master')
+ if os.path.isfile(file_path):
+ with open(file_path, 'r') as hash_file:
+ hsh = hash_file.read().strip(' \n')
+ rev, date = getversion_git_windows(hsh, path)
+ else:
+ raise ParseError
tag = open(os.path.join(_program_dir, '.git/config'), 'r').read()
s = tag.find('url = ', tag.find('[remote "origin"]'))
e = tag.find('\n', s)
--
To view, visit
https://gerrit.wikimedia.org/r/203305
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa6458e60e42bf3026ff56af9e797ec580e6bb85
Gerrit-PatchSet: 4
Gerrit-Project: pywikibot/compat
Gerrit-Branch: master
Gerrit-Owner: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: XZise <CommodoreFabianus(a)gmx.de>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>