Merge pull request #1962 from effigies/bug/fsversion

FIX: Restore FSVersion to string

Author: chrisgorgo

nipype/interfaces/freesurfer/preprocess.py

@@ -38,7 +38,7 @@
 
 # Keeping this to avoid breaking external programs that depend on it, but
 # this should not be used internally
-FSVersion = Info.looseversion()
+FSVersion = Info.looseversion().vstring
 
 
 class ParseDICOMDirInputSpec(FSTraitedSpec):

nipype/interfaces/freesurfer/tests/test_preprocess.py

@@ -1,6 +1,7 @@
 # -*- coding: utf-8 -*-
 # emacs: -*- mode: python; py-indent-offset: 4; indent-tabs-mode: nil -*-
 # vi: set ft=python sts=4 ts=4 sw=4 et:
+from builtins import str
 import os
 
 import pytest
@@ -154,3 +155,9 @@ def test_bbregister(create_files_in_directory):
                            '--reg {base}_bbreg_fsaverage.dat '
                            '--mov {full} --s fsaverage'.format(
                             full=filelist[0], base=base))
+
+def test_FSVersion():
+    """Check that FSVersion is a string that can be compared with LooseVersion
+    """
+    assert isinstance(freesurfer.preprocess.FSVersion, str)
+    assert LooseVersion(freesurfer.preprocess.FSVersion) >= LooseVersion("0")