/Nim

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ansi color code templates fail to bind symbols #9394

Closed
bluenote10 opened this Issue Oct 16, 2018 · 0 comments

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@bluenote10
@bluenote10
Copy link
Contributor

bluenote10 commented Oct 16, 2018

The code

import terminal, colors
let code = ansiForegroundColorCode(colAliceBlue)

errors with

../home/fabian/bin/nim-repo/lib/pure/terminal.nim(639, 14) template/generic instantiation from here
../home/fabian/bin/nim-repo/lib/pure/strformat.nim(315, 39) Error: undeclared identifier: 'fgPrefix'

Looking at the source shows that it is probably a consequence of using stformat in a template.

Vindaar added a commit to Vindaar/Nim that referenced this issue Oct 17, 2018

@Vindaar
fix nim-lang#9394 by replacing `fmt` with normal string append 
Until issue nim-lang#7632 is fixed, use string append.
fe15da9

@Vindaar Vindaar referenced this issue Oct 17, 2018

Merged

fix #9394 by replacing `fmt` with `strutils.%` #9417

@Araq Araq closed this in #9417 Oct 18, 2018

Araq added a commit that referenced this issue Oct 18, 2018

@Vindaar @Araq
fix #9394 by replacing `fmt` with `strutils.%` (#9417) 
* fix #9394 by replacing `fmt` with normal string append

Until issue #7632 is fixed, use string append.

* use `strutils.%` instead of normal string add
Loading status checks…
82a1576

narimiran added a commit to narimiran/Nim that referenced this issue Oct 31, 2018

@Vindaar @narimiran
fix nim-lang#9394 by replacing `fmt` with `strutils.%` (nim-lang#9417) 
* fix nim-lang#9394 by replacing `fmt` with normal string append

Until issue nim-lang#7632 is fixed, use string append.

* use `strutils.%` instead of normal string add
8c0a86c

narimiran added a commit to narimiran/Nim that referenced this issue Nov 1, 2018

@Vindaar @narimiran
fix nim-lang#9394 by replacing `fmt` with `strutils.%` (nim-lang#9417) 
* fix nim-lang#9394 by replacing `fmt` with normal string append

Until issue nim-lang#7632 is fixed, use string append.

* use `strutils.%` instead of normal string add
d12c745

narimiran added a commit that referenced this issue Nov 1, 2018

@Vindaar @narimiran
fix #9394 by replacing `fmt` with `strutils.%` (#9417) 
* fix #9394 by replacing `fmt` with normal string append

Until issue #7632 is fixed, use string append.

* use `strutils.%` instead of normal string add
5017f7c

narimiran added a commit that referenced this issue Nov 1, 2018

@Vindaar @narimiran
fix #9394 by replacing `fmt` with `strutils.%` (#9417) 
* fix #9394 by replacing `fmt` with normal string append

Until issue #7632 is fixed, use string append.

* use `strutils.%` instead of normal string add

(cherry picked from commit 82a1576)
58726f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment