Update to support Nim 0.19 (removal of string nil specifically).
This commit is contained in:
parent
e0eb8fd4c2
commit
6fcb8e8e1e
28
cliutils.nim
28
cliutils.nim
@ -41,20 +41,20 @@ type HandleProcMsgCB* = proc (outMsg: TaintedString,
|
|||||||
errMsg: TaintedString, cmd: string): void
|
errMsg: TaintedString, cmd: string): void
|
||||||
|
|
||||||
proc sendMsg*(h: HandleProcMsgCB, outMsg: TaintedString,
|
proc sendMsg*(h: HandleProcMsgCB, outMsg: TaintedString,
|
||||||
errMsg: TaintedString = nil, cmd: string = ""): void =
|
errMsg: TaintedString = "", cmd: string = ""): void =
|
||||||
if h != nil: h(outMsg, errMsg, cmd)
|
if h != nil: h(outMsg, errMsg, cmd)
|
||||||
|
|
||||||
proc makeProcMsgHandler*(outSink, errSink: File, prefixCmd: bool = true): HandleProcMsgCB =
|
proc makeProcMsgHandler*(outSink, errSink: File, prefixCmd: bool = true): HandleProcMsgCB =
|
||||||
result = proc(outMsg, errMsg: TaintedString, cmd: string): void {.closure.} =
|
result = proc(outMsg, errMsg: TaintedString, cmd: string): void {.closure.} =
|
||||||
let prefix = if cmd == nil or not prefixCmd: "" else: cmd & ": "
|
let prefix = if cmd.len == 0 or not prefixCmd: "" else: cmd & ": "
|
||||||
if outMsg != nil: outSink.writeLine(prefix & outMsg)
|
if outMsg.len > 0: outSink.writeLine(prefix & outMsg)
|
||||||
if errMsg != nil: errSink.writeLine(prefix & errMsg)
|
if errMsg.len > 0: errSink.writeLine(prefix & errMsg)
|
||||||
|
|
||||||
proc makeProcMsgHandler*(outSink, errSink: Stream, prefixCmd: bool = true): HandleProcMsgCB =
|
proc makeProcMsgHandler*(outSink, errSink: Stream, prefixCmd: bool = true): HandleProcMsgCB =
|
||||||
result = proc(outMsg, errMsg: TaintedString, cmd: string): void {.closure.} =
|
result = proc(outMsg, errMsg: TaintedString, cmd: string): void {.closure.} =
|
||||||
let prefix = if cmd == nil or not prefixCmd: "" else: cmd & ": "
|
let prefix = if cmd.len == 0 or not prefixCmd: "" else: cmd & ": "
|
||||||
if outMsg != nil: outSink.writeLine(prefix & outMsg)
|
if outMsg.len > 0: outSink.writeLine(prefix & outMsg)
|
||||||
if errMsg != nil: errSink.writeLine(prefix & errMsg)
|
if errMsg.len > 0: errSink.writeLine(prefix & errMsg)
|
||||||
|
|
||||||
proc combineProcMsgHandlers*(a, b: HandleProcMsgCB): HandleProcMsgCB =
|
proc combineProcMsgHandlers*(a, b: HandleProcMsgCB): HandleProcMsgCB =
|
||||||
if a == nil: result = b
|
if a == nil: result = b
|
||||||
@ -72,9 +72,9 @@ proc waitFor*(p: Process, msgCB: HandleProcMsgCB, procCmd: string = ""): int =
|
|||||||
var line = newStringOfCap(120).TaintedString
|
var line = newStringOfCap(120).TaintedString
|
||||||
while true:
|
while true:
|
||||||
if pout.readLine(line):
|
if pout.readLine(line):
|
||||||
msgCB.sendMsg(line, nil, procCmd)
|
msgCB.sendMsg(line, "", procCmd)
|
||||||
elif perr.readLine(line):
|
elif perr.readLine(line):
|
||||||
msgCB.sendMsg(nil, line, procCmd)
|
msgCB.sendMsg("", line, procCmd)
|
||||||
else:
|
else:
|
||||||
result = peekExitCode(p)
|
result = peekExitCode(p)
|
||||||
if result != -1: break
|
if result != -1: break
|
||||||
@ -100,8 +100,8 @@ proc execWithOutput*(command: string, workingDir:string = "",
|
|||||||
outSeq = @[]; errSeq = @[]
|
outSeq = @[]; errSeq = @[]
|
||||||
var outputCollector = combineProcMsgHandlers(msgCB,
|
var outputCollector = combineProcMsgHandlers(msgCB,
|
||||||
proc(outMsg, errMsg: TaintedString, cmd: string): void {.closure.} =
|
proc(outMsg, errMsg: TaintedString, cmd: string): void {.closure.} =
|
||||||
if outMsg != nil: outSeq.add(outMsg)
|
if outMsg.len > 0: outSeq.add(outMsg)
|
||||||
if errMsg != nil: errSeq.add(errMsg))
|
if errMsg.len > 0: errSeq.add(errMsg))
|
||||||
|
|
||||||
result[2] = exec(command, workingDir, args, env, options, outputCollector)
|
result[2] = exec(command, workingDir, args, env, options, outputCollector)
|
||||||
result[0] = outSeq.join("\n")
|
result[0] = outSeq.join("\n")
|
||||||
@ -147,15 +147,15 @@ proc daemonize*(pidfile, si, so, se: string, daemonMain: proc(): void): Pid =
|
|||||||
flushFile(stdout)
|
flushFile(stdout)
|
||||||
flushFile(stderr)
|
flushFile(stderr)
|
||||||
|
|
||||||
if not si.isNil and si != "":
|
if si.len > 0:
|
||||||
fi = open(si, fmRead)
|
fi = open(si, fmRead)
|
||||||
discard dup2(getFileHandle(fi), getFileHandle(stdin))
|
discard dup2(getFileHandle(fi), getFileHandle(stdin))
|
||||||
|
|
||||||
if not so.isNil and so != "":
|
if so.len > 0:
|
||||||
fo = open(so, fmAppend)
|
fo = open(so, fmAppend)
|
||||||
discard dup2(getFileHandle(fo), getFileHandle(stdout))
|
discard dup2(getFileHandle(fo), getFileHandle(stdout))
|
||||||
|
|
||||||
if not se.isNil and so != "":
|
if se.len > 0:
|
||||||
fe = open(se, fmAppend)
|
fe = open(se, fmAppend)
|
||||||
discard dup2(getFileHandle(fe), getFileHandle(stderr))
|
discard dup2(getFileHandle(fe), getFileHandle(stderr))
|
||||||
|
|
||||||
|
@ -1,11 +1,11 @@
|
|||||||
# Package
|
# Package
|
||||||
|
|
||||||
version = "0.5.0"
|
version = "0.6.0"
|
||||||
author = "Jonathan Bernard"
|
author = "Jonathan Bernard"
|
||||||
description = "Helper functions for writing command line interfaces."
|
description = "Helper functions for writing command line interfaces."
|
||||||
license = "MIT"
|
license = "MIT"
|
||||||
|
|
||||||
# Dependencies
|
# Dependencies
|
||||||
|
|
||||||
requires @["nim >= 0.18.0", "docopt"]
|
requires @["nim >= 0.19.0", "docopt >= 0.6.8"]
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user