DrTrigon has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/81196


Change subject: bug fix and revert for version handling
......................................................................

bug fix and revert for version handling

* git version info extraction was buggy
* version handling was messed up by change of __version__ to __initversion__ 
(use __release__ instead!!)

Change-Id: Ibbf5953bf8759f94323f9d31bded2c3095ba58a0
---
M pywikibot/__init__.py
M pywikibot/version.py
M scripts/version.py
3 files changed, 8 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core 
refs/changes/96/81196/1

diff --git a/pywikibot/__init__.py b/pywikibot/__init__.py
index 27c9d52..414550d 100644
--- a/pywikibot/__init__.py
+++ b/pywikibot/__init__.py
@@ -7,8 +7,8 @@
 #
 # Distributed under the terms of the MIT license.
 #
-__version__ = '2.0b1'
-__initversion__ = '$Id$'
+__release__ = '2.0b1'
+__version__ = '$Id$'
 
 import datetime
 import difflib
diff --git a/pywikibot/version.py b/pywikibot/version.py
index 6a8eb92..286e0b7 100644
--- a/pywikibot/version.py
+++ b/pywikibot/version.py
@@ -10,8 +10,7 @@
 __version__ = '$Id$'
 
 import os
-import time
-import datetime
+import time, datetime
 import urllib
 import subprocess
 
@@ -121,6 +120,8 @@
     tag = tag[(s + 6):e]
     t = tag.strip().split('/')
     tag = '[%s] %s' % (t[0][:-1], '/'.join(t[3:])[:-4])
+    path = os.path.abspath(os.curdir)
+    os.chdir(_program_dir)
     info = subprocess.Popen("git log --pretty=format:'%ad|%an|%h|%H|%d' 
--abbrev-commit --date=iso -1 | cat -",
                             shell=True,
                             stdout=subprocess.PIPE).stdout.read()
@@ -130,6 +131,7 @@
     rev = subprocess.Popen('git rev-list HEAD | wc -l',
                            shell=True,
                            stdout=subprocess.PIPE).stdout.read()
+    os.chdir(path)
     rev = int(rev.strip())
     hsh = info[3]      # also stored in '.git/refs/heads/master'
     if (not date or not tag or not rev) and not path:
@@ -173,7 +175,7 @@
     fn = os.path.join(_program_dir, filename)
     if os.path.exists(fn):
         for line in open(fn, 'r').readlines():
-            if line.find('__initversion__') == 0:
+            if line.find('__version__') == 0:
                 exec(line)
                 break
         stat = os.stat(fn)
diff --git a/scripts/version.py b/scripts/version.py
index 32b06df..7cb2bc7 100755
--- a/scripts/version.py
+++ b/scripts/version.py
@@ -17,7 +17,7 @@
 
 if __name__ == '__main__':
     print 'Pywikibot: %s' % getversion()
-    print 'Release version: %s' % pywikibot.__version__
+    print 'Release version: %s' % pywikibot.__release__
     print 'Python: %s' % sys.version
     #print 'config-settings:'
     #print 'site_interface =', config.site_interface

-- 
To view, visit https://gerrit.wikimedia.org/r/81196
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibbf5953bf8759f94323f9d31bded2c3095ba58a0
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: DrTrigon <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to