From 191b11e97bb6f82b0c08ce001000f243c3d35ad1 Mon Sep 17 00:00:00 2001
From: Kelly Sovacool <kelly.sovacool@nih.gov>
Date: Thu, 2 Jan 2025 14:04:03 -0500
Subject: [PATCH] fix(runlocal): only check slurm job id on biowulf and frce

to enforce interactive node only
---
 charlie | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/charlie b/charlie
index 594ff11..a712c9c 100755
--- a/charlie
+++ b/charlie
@@ -338,8 +338,6 @@ function printbinds(){
 
 function runlocal() {
   runcheck
-  # TODO do not assume $SLURM_JOB_ID exists, may run on other platform without slurm e.g. eddie
-  if [ "$SLURM_JOB_ID" == "" ];then err "runlocal can only be done on an interactive node"; exit 1; fi
   run "local"
 }
 
@@ -431,6 +429,12 @@ function run() {
 
   if [ "$1" == "local" ];then
 
+    if [ "$PLATFORM" == "biowulf" ] || [ "$PLATFORM" == "fnlcr"]; then
+      if [ "$SLURM_JOB_ID" == "" ]; then
+        err "runlocal can only be done on an interactive node";
+        exit 1;
+      fi
+    fi
     preruncleanup
 
     $EXPORT_SING_CACHE_DIR_CMD