Merge pull request #76 from broadinstitute/ccle_portal_hotfix

v0.16.10: hotfix for robust deployment on Google AppEngine

changelog.txt

@@ -2,6 +2,8 @@
 Change Log for FISSFC: the (Fi)recloud (S)ervice (S)elector
 =======================================================================
 Terms used below:  HL = high level interface, LL = low level interface
+v0.16.9 -  Hotfix; modified how subprocess.call() is utilized to avoid
+           problems on Google AppEngine, where it is prohibited
 v0.16.8 -  Hotfix; entity_import corrected to only strip newline characters.
 
 v0.16.7 -  Enabled LL and HL access to the linkExistingEntities field of the

firecloud/fccore.py

@@ -21,7 +21,7 @@
 import tempfile
 import shutil
 from collections import Iterable
-from subprocess import call
+import subprocess
 from io import IOBase
 from firecloud import __about__
 from google.auth import environment_vars
@@ -193,7 +193,7 @@ def edit_text(text=None):
     if text:
         tf.write(text)
         tf.flush()
-    call([__EDITOR__, tf.name])
+    subprocess.call([__EDITOR__, tf.name])
     with open(tf.name, 'r') as newfile:
         text = newfile.read()
     tf.close()
@@ -211,7 +211,7 @@ def edit_file(name, backup=None):
         shutil.copy2(name, backup)
     # Record time of last modification: tolm
     previous_tolm = os.stat(name).st_mtime
-    call([__EDITOR__, name])
+    subprocess.call([__EDITOR__, name])
     current_tolm  = os.stat(name).st_mtime
     return current_tolm != previous_tolm