fix nim-lang#14142: no more clash with: import os + use of existsDir/…

…dirExists/existsFile/fileExists/findExe in config.nims
timotheecour · Apr 29, 2020 · 75ac5f8 · 75ac5f8
1 parent 3b5a504
commit 75ac5f8
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/compiler/condsyms.nim b/compiler/condsyms.nim
@@ -77,7 +77,7 @@ proc initDefines*(symbols: StringTableRef) =
   defineSymbol("nimHasUserErrors")
   defineSymbol("nimUncheckedArrayTyp")
   defineSymbol("nimHasTypeof")
-  defineSymbol("nimErrorProcCanHaveBody")
+  defineSymbol("nimErrorProcCanHaveBody") # since #9665 but now unused
   defineSymbol("nimHasInstantiationOfInMacro")
   defineSymbol("nimHasHotCodeReloading")
   defineSymbol("nimHasNilSeqs")

diff --git a/lib/pure/os.nim b/lib/pure/os.nim
@@ -74,8 +74,10 @@ elif defined(posix):
 else:
   {.error: "OS module not ported to your operating system!".}
 
-when weirdTarget and defined(nimErrorProcCanHaveBody):
-  {.pragma: noNimScript, error: "this proc is not available on the NimScript target".}
+when weirdTarget:
+  template noNimScript(body): untyped = discard
+  # Adding a `disable` template and `{.pragma: noNimScript, disable.}`
+  # doesn't work pending https://github.com/timotheecour/Nim/issues/142
 else:
   {.pragma: noNimScript.}
 

diff --git a/tests/test_nimscript.nims b/tests/test_nimscript.nims
@@ -69,8 +69,14 @@ import std/[
   decls, compilesettings, with, wrapnils
 ]
 
-
 block:
   doAssert "./foo//./bar/".normalizedPath == "foo/bar".unixToNativePath
 
 echo "Nimscript imports are successful."
+
+block: # #14142
+  discard existsDir("/usr")
+  discard dirExists("/usr")
+  discard existsFile("/usr/foo")
+  discard fileExists("/usr/foo")
+  discard findExe("nim")