Merge pull request #220 from NOAA-GFDL/212.make-scripts-executable

#212 Make compile and checkout scripts executable
NOAA-GFDL · Oct 28, 2024 · 65fc0c2 · 65fc0c2
2 parents d1457e2 + be01922
commit 65fc0c2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/fre/make/gfdlfremake/buildBaremetal.py b/fre/make/gfdlfremake/buildBaremetal.py
@@ -110,6 +110,9 @@ def writeScript(self):
         self.f.write(self.make+"\n")
         self.f.close()
 
+        # Make compile script executable
+        os.chmod(self.bld+"/compile.sh", 0o744)
+
 ## TODO run as a batch job on the login cluster
     def run(self):
         """
@@ -118,7 +121,6 @@ def run(self):
             - self : The dockerfile object
         """
 ###### TODO make the Makefile
-        os.chmod(self.bld+"/compile.sh", 0o744)
         command = [self.bld+"/compile.sh","|","tee",self.bld+"/log.compile"]
         try:
             subprocess.run(args=command, check=True)

diff --git a/fre/make/gfdlfremake/checkout.py b/fre/make/gfdlfremake/checkout.py
@@ -108,14 +108,16 @@ def finish (self,pc):
         else:
             self.checkoutScript.close()
 
+        # Make checkout script executable
+        os.chmod(self.src+"/"+self.fname, 0o744)
+
 ## TODO: batch script building
     def run (self):
         """
         Brief: Changes the permission on the checkout script and runs it
         Param:
             - self The checkout script object
         """
-        os.chmod(self.src+"/"+self.fname, 0o744)
         try:
             subprocess.run(args=[self.src+"/"+self.fname], check=True)
         except: