Merge pull request #2542 from djarecka/fixget_info_2541

using RawConfigParser instead of ConfigParser (closes #2541)

Author: satra

nipype/pkg_info.py

@@ -49,10 +49,7 @@ def pkg_commit_hash(pkg_path):
     pth = os.path.join(pkg_path, COMMIT_INFO_FNAME)
     if not os.path.isfile(pth):
         raise IOError('Missing commit info file %s' % pth)
-    if PY3:
-        cfg_parser = configparser.RawConfigParser()
-    else:
-        cfg_parser = configparser.ConfigParser()
+    cfg_parser = configparser.RawConfigParser()
     with open(pth, encoding='utf-8') as fp:
         cfg_parser.readfp(fp)
     archive_subst = cfg_parser.get('commit hash', 'archive_subst_hash')

nipype/tests/test_nipype.py

@@ -1,4 +1,6 @@
 from .. import get_info
+from ..info import get_nipype_gitversion
+import pytest
 
 
 def test_nipype_info():
@@ -10,5 +12,8 @@ def test_nipype_info():
     assert exception_not_raised
 
 
-# def test_fail_always():
-#     assert False
+@pytest.mark.skipif(not get_nipype_gitversion(), 
+                    reason="not able to get version from get_nipype_gitversion")
+def test_git_hash():
+    # removing the first "g" from gitversion
+    get_nipype_gitversion()[1:] == get_info()['commit_hash']

setup.py

@@ -76,10 +76,7 @@ def run(self):
             repo_commit = repo_commit.decode()
 
         # We write the installation commit even if it's empty
-        if PY3:
-            cfg_parser = configparser.RawConfigParser()
-        else:
-            cfg_parser = configparser.ConfigParser()
+        cfg_parser = configparser.RawConfigParser()
         cfg_parser.read(pjoin('nipype', 'COMMIT_INFO.txt'))
         cfg_parser.set('commit hash', 'install_hash', repo_commit.strip())
         out_pth = pjoin(self.build_lib, 'nipype', 'COMMIT_INFO.txt')