Minor code simplifications/clarifications

src/Pigeons.jl

@@ -60,7 +60,7 @@ import Base.merge
 
 import DynamicPPL
 
-const use_auto_exec_folder = ""
+const use_auto_exec_folder = "use_auto_exec_folder"
 
 include("includes.jl")
 

src/utils/exec_folder.jl

@@ -31,20 +31,15 @@ function safelink(target::AbstractString, link::AbstractString)
     try
         symlink(relative_path, link)
     catch e # on windows, need admin to do symlink (!)
-        if !_safelink_warned[]
-            println("""
-                WARN: could not create symlink($relative_path, $link)
+        @warn   """
+                Could not create symlink($relative_path, $link)
                     If you are running windows, this is a known issue, 
                     you will need to run under admin permission, see: 
                     https://discourse.julialang.org/t/symlink-can-not-create-hardlink-in-windows10/75702
                     For now, skipping symlinks, this may affect some features 
                     such as loading checkpoints. 
                 Details: 
                 $e
-                """)
-            _safelink_warned[] = true
-        end
+                """ maxlog=1
     end
 end
-
-const _safelink_warned::Ref{Bool} = Ref(false)

src/utils/misc.jl

@@ -125,12 +125,3 @@ function sort_includes(main)
         error(msg)
     end
 end
-
-function sh(script::AbstractString)
-    path, io = mktemp()
-    write(io, script)
-    close(io)
-    result = read(`sh $path`, String)
-    println(result)
-    return result
-end