Making sure *nix systems are running with shell=False

DICOMtools/loadDicom.py

@@ -415,15 +415,25 @@ def isDicom(file):
     return dt=='DICM'
 
 
-def dicomValue(file,tag,seqTag=None,shell=False):
+def dicomValue(fileName,tag,seqTag=None,shell=False):
     #query dicom value of file using dcmdump (DCMTK routine)
+
+    #shell must be false for *nix
+    if os.name=="posix":
+        shell=False
+
+    debug=False
     dcmdump=os.path.join(os.environ['SLICER_HOME'],"bin","dcmdump")
+    
     try:
-        out=subprocess.check_output([dcmdump,'+p','+P',tag,file],shell=shell)
+        if debug:
+            print("Calling {}".format(dcmdump))
+        out=subprocess.check_output([dcmdump,'+p','+P',tag,fileName],shell=shell)
+        if debug:
+            print("Got {}".format(out))
     except subprocess.CalledProcessError as e:
         return None
     
-    debug=False
     
     if debug:
         print("Tag {} Line '{}'").format(tag,out)