Only print the link command when listCmd is active; fix docs (#13603)

nim-lang · Mar 7, 2020 · eae31a7 · eae31a7
1 parent 6a5b381
commit eae31a7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/compiler/extccomp.nim b/compiler/extccomp.nim
@@ -489,7 +489,7 @@ proc addExternalFileToLink*(conf: ConfigRef; filename: AbsoluteFile) =
   conf.externalToLink.insert(filename.string, 0)
 
 proc execWithEcho(conf: ConfigRef; cmd: string, msg = hintExecuting): int =
-  rawMessage(conf, msg, cmd)
+  rawMessage(conf, msg, if msg == hintLinking and not(optListCmd in conf.globalOptions or conf.verbosity > 1): "" else: cmd)
   result = execCmd(cmd)
 
 proc execExternalProgram*(conf: ConfigRef; cmd: string, msg = hintExecuting) =

diff --git a/compiler/nim.cfg b/compiler/nim.cfg
@@ -1,7 +1,6 @@
 # Special configuration file for the Nim project
 
 hint[XDeclaredButNotUsed]:off
-hint[Link]:off
 
 define:booting
 define:nimcore

diff --git a/doc/advopt.txt b/doc/advopt.txt
@@ -125,8 +125,8 @@ Advanced options:
                             dynlib: "liblua.so.3"
   --dynlibOverrideAll
                             disables the effects of the dynlib pragma
-  --listCmd                 list the compilation commands; can be combined with:
-                            --hint:exec:on and --hint:link:on
+  --listCmd                 list the compilation commands; can be combined with
+                            `--hint:exec:on` and `--hint:link:on`
   --asm                     produce assembler code
   --parallelBuild:0|1|...   perform a parallel build
                             value = number of processors (0 for auto-detect)