jdb/daily-notifier
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Workaround for Nim bug 4996

Browse Source 
As of Nim 0.15.2 there exists a conflict the compiler cannot reconcile between
`sequtils.toSeq` and `nre.toSeq`. The workaround is to be explicit about which
`toSeq` you want (see https://github.com/nim-lang/Nim/issues/4996).
This commit is contained in:
Jonathan Bernard 2016-11-05 08:17:06 -05:00
parent ddf9ab7fd1
commit 3e430f8ff2
2 changed files with 8 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
daily_notifier.nim
View File

@ -1,4 +1,4 @@
import cliutils, docopt, json, logging, os, re, random, sequtils, import cliutils, docopt, json, logging, os, nre, random, sequtils,
times, timeutils times, timeutils
from posix import SIGTERM, SIGHUP, signal, kill from posix import SIGTERM, SIGHUP, signal, kill
@ -11,7 +11,7 @@ type
DPConfig = tuple[planDir, dateFmt, pidfile, logfile, errfile: string, DPConfig = tuple[planDir, dateFmt, pidfile, logfile, errfile: string,
notificationSecs: int] notificationSecs: int]
const VERSION = "0.3.0" const VERSION = "0.3.1"
const NOTE_TITLE = "Daily Notifier v" & VERSION const NOTE_TITLE = "Daily Notifier v" & VERSION
const timeFmt = "HH:mm" const timeFmt = "HH:mm"
@ -19,7 +19,7 @@ var args: Table[string, Value]
var cfg: DPConfig var cfg: DPConfig
let appName = getAppFilename() let appName = getAppFilename()
let soundsDir: string = appName[0..(appName.len-15)] & "sounds/navi" let soundsDir: string = appName[0..(appName.len-15)] & "sounds/navi"
let soundFiles = toSeq(walkFiles(soundsDir & "/*")) let soundFiles = seqUtils.toSeq(walkFiles(soundsDir & "/*"))
randomize() randomize()
proc parseDailyPlan(filename: string): seq[PlanItem] = proc parseDailyPlan(filename: string): seq[PlanItem] =
@ -40,22 +40,10 @@ proc parseDailyPlan(filename: string): seq[PlanItem] =
if line.strip.startsWith("## Timeline"): parseState = ReadingPlans if line.strip.startsWith("## Timeline"): parseState = ReadingPlans
of ReadingPlans: of ReadingPlans:
# TODO: This is the better code using the nre module: let match = line.find(planItemRe)
# let match = line.find(planItemRe) if match.isSome(): result.add((
# if match.isSome(): result.add(( time: parse(match.get().captures[0], timeFmt),
# time: parse(match.get().captures[0], timeFmt), note: match.get().captures[1]))
# note: match.get().captures[1]))
# else: parseState = AfterPlans
#
# Curently there is an incompatibility between the os and nre modules
# (see https://github.com/nim-lang/Nim/issues/4996) so the following code
# is used to avoid using nre until the bug is fixed.
if line.match(planItemRe):
let stripped = line.strip
result.add((
time: parse(stripped[0..4], timeFmt),
note: stripped.substr(5)))
else: parseState = AfterPlans else: parseState = AfterPlans
of AfterPlans: break of AfterPlans: break

2
daily_notifier.nimble
View File

@ -1,6 +1,6 @@
# Package # Package
version = "0.3.0" version = "0.3.1"
author = "Jonathan Bernard" author = "Jonathan Bernard"
description = "Little programs that reads my daily plan and notifies me of upcoming events." description = "Little programs that reads my daily plan and notifies me of upcoming events."
license = "MIT" license = "MIT"