jenkins-bot has submitted this change and it was merged. Change subject: (bug 59963) ignore git not found errors ......................................................................
(bug 59963) ignore git not found errors By calling 'git' directly instead of via a shell, an Exception is raised instead of an error being printed. Change-Id: Ide8b1c5407ce58243ac4615ae450e0ccf5132099 --- M pywikibot/version.py 1 file changed, 14 insertions(+), 5 deletions(-) Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified diff --git a/pywikibot/version.py b/pywikibot/version.py index 6dd9514..13b86b4 100644 --- a/pywikibot/version.py +++ b/pywikibot/version.py @@ -112,6 +112,13 @@ def getversion_git(path=None): _program_dir = path or _get_program_dir() + cmd = 'git' + try: + subprocess.Popen([cmd], stdout=subprocess.PIPE).communicate() + except WindowsError: + # some windows git versions provide git.cmd instead of git.exe + cmd = 'git.cmd' + #(try to use .git directory for new entries format) #tag = subprocess.Popen('git config --get remote.origin.url', # shell=True, @@ -122,16 +129,18 @@ tag = tag[(s + 6):e] t = tag.strip().split('/') tag = '[%s] %s' % (t[0][:-1], '/'.join(t[3:])[:-4]) - info = subprocess.Popen('git --no-pager log --pretty=format:"%ad|%an|%h|%H|%d" --abbrev-commit --date=iso -1', + info = subprocess.Popen([cmd, '--no-pager', + 'log', '-1', + '--pretty=format:"%ad|%an|%h|%H|%d"' + '--abbrev-commit', + '--date=iso'], cwd=_program_dir, - shell=True, stdout=subprocess.PIPE).stdout.read() info = info.split('|') date = info[0][:-6] - date = time.strptime(date, '%Y-%m-%d %H:%M:%S') - rev = subprocess.Popen('git rev-list HEAD', + date = time.strptime(date.strip('"'), '%Y-%m-%d %H:%M:%S') + rev = subprocess.Popen([cmd, 'rev-list', 'HEAD'], cwd=_program_dir, - shell=True, stdout=subprocess.PIPE).stdout.read() rev = len(rev.splitlines()) hsh = info[3] # also stored in '.git/refs/heads/master' -- To view, visit https://gerrit.wikimedia.org/r/107006 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide8b1c5407ce58243ac4615ae450e0ccf5132099 Gerrit-PatchSet: 5 Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-Owner: Merlijn van Deen <valhall...@arctus.nl> Gerrit-Reviewer: Ladsgroup <ladsgr...@gmail.com> Gerrit-Reviewer: Merlijn van Deen <valhall...@arctus.nl> Gerrit-Reviewer: Xqt <i...@gno.de> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits