INim icon indicating copy to clipboard operation
INim copied to clipboard

Doesn't compile with 1.6.0 on macOS

Open ynfle opened this issue 2 years ago • 3 comments

 $ nimble install inim --verbose
    Reading official package list
Downloading https://github.com/inim-repl/INim using git
    Cloning latest tagged version: v0.6.1
  Verifying dependencies for [email protected]
    Reading official package list
   Checking for cligen@>= 1.2.0
      Info: Dependency on cligen@>= 1.2.0 already satisfied
  Verifying dependencies for [email protected]
    Reading official package list
   Checking for noise@>= 0.1.4
      Info: Dependency on noise@>= 0.1.4 already satisfied
  Verifying dependencies for [email protected]
 Installing [email protected]
   Building inim/inim using c backend
  Executing ~/.nimble/bin/nim c --colors:on --noNimblePath -d:release -d:NimblePkgVersion=0.6.1 --path:~/.nimble/pkgs/cligen-1.2.2 --path:~/.nimble/pkgs/noise-0.1.4 -o:/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/inim /var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim
Hint: used config file '~/.choosenim/toolchains/nim-1.6.0/config/nim.cfg' [Conf]
Hint: used config file '~/.choosenim/toolchains/nim-1.6.0/config/config.nims' [Conf]
......................................................................................................
/private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(5, 15) Hint: duplicate import of 'strformat'; previous import here: /private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(4, 20) [DuplicateModuleImport]
....................
/private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(7, 1) Hint: duplicate import of 'sequtils'; previous import here: /private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(4, 51) [DuplicateModuleImport]
.
/private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(420, 9) Hint: 'n' is declared but not used [XDeclaredButNotUsed]
/private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(457, 11) Hint: 'n' is declared but not used [XDeclaredButNotUsed]
/private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(543, 21) Hint: 'echo_status' is declared but not used [XDeclaredButNotUsed]
................
/private/var/folders/st/xh33tkfx0x329ltvx1c4j78m0000gn/T/nimble_49023/githubcom_inimreplINim/src/inim.nim(627, 11) template/generic instantiation of `dispatch` from here
~/.nimble/pkgs/cligen-1.2.2/cligen.nim(768, 14) template/generic instantiation of `dispatchCf` from here
~/.nimble/pkgs/cligen-1.2.2/cligen.nim(745, 31) Error: expression 'dispatchmain(mergeParams([""], commandLineParams()), clUse, "", false, false,
             false)' has no type (or is ambiguous)
 $ nim -v
Nim Compiler Version 1.6.0 [MacOSX: amd64]
Compiled at 2021-10-19
Copyright (c) 2006-2021 by Andreas Rumpf

active boot switches: -d:release

ynfle avatar Oct 19 '21 23:10 ynfle

Same error on Windows 10, Nim 1.6.0.

auxym avatar Dec 16 '21 18:12 auxym

Can't replicate, could you confirm on the latest version of nim?

λ choosenim update stable                                            10:42:11
   Updating stable
Downloading Nim 1.6.4 from nim-lang.org
[##################################################] 100.0% 0kb/s
 Extracting nim-1.6.4-linux_x64.tar.xz
 Extracting nim-1.6.4-linux_x64.tar
   Building Nim 1.6.4
  Compiler: Already built
    Updated to 1.6.4
   Switched to Nim 1.6.4
~ via  v12.22.1 via  v1.61.0-nightly 
λ nimble install inim                                                10:42:23
Downloading https://github.com/inim-repl/INim using git
  Verifying dependencies for [email protected]
      Info: Dependency on cligen@>= 1.2.0 already satisfied
  Verifying dependencies for [email protected]
      Info: Dependency on noise@>= 0.1.4 already satisfied
  Verifying dependencies for [email protected]
 Installing [email protected]
   Building inim/inim using c backend
    Prompt: [email protected] already exists. Overwrite? [y/N]
y
    Answer:    Success: inim installed successfully.

0atman avatar Mar 30 '22 09:03 0atman

Only some versions of cligen exhibit the issue, at least one of which is v 1.3.2:

❯ choosenim update stable
   Updating stable
      Info: Already up to date at version 1.6.4

~
❯ nim --version
Nim Compiler Version 1.6.4 [Windows: amd64]
Compiled at 2022-02-09
Copyright (c) 2006-2021 by Andreas Rumpf

active boot switches: -d:release

❯ nimble install [email protected]
Downloading https://github.com/c-blake/cligen.git using git
  Verifying dependencies for [email protected]
 Installing [email protected]
   Success: cligen installed successfully.

~ took 4s
❯ nimble install inim
Downloading https://github.com/AndreiRegiani/INim using git
  Verifying dependencies for [email protected]
      Info: Dependency on cligen@>= 1.2.0 already satisfied
  Verifying dependencies for [email protected]
      Info: Dependency on noise@>= 0.1.4 already satisfied
  Verifying dependencies for [email protected]
 Installing [email protected]
   Building inim/inim.exe using c backend
C:\Users\thef2102\AppData\Local\Temp\nimble_17068\githubcom_AndreiRegianiINim\src\inim.nim(627, 11) template/generic instantiation of `dispatch` from here
C:\Users\thef2102\.nimble\pkgs\cligen-1.3.2\cligen.nim(777, 14) template/generic instantiation of `dispatchCf` from here
C:\Users\thef2102\.nimble\pkgs\cligen-1.3.2\cligen.nim(754, 31) Error: expression 'dispatchmain(mergeParams([""], commandLineParams()), clUse, "", false, false,
             false)' has no type (or is ambiguous)
    Prompt: Build failed for '[email protected]', would you like to try installing 'inim@#head' (latest unstable)? [y/N]
    Answer: n
       Tip: 8 messages have been suppressed, use --verbose to show them.
     Error: Aborting installation due to build failure

[email protected] satisfies inim's (0.6.1) constraint of `cligen>=1.2.0, therefore users who already have these "broken" cligen versions on their system would result in a failed build for inim.

#131 should resolve this issue, we just need to issue a new release (tag) of inim.

auxym avatar Mar 30 '22 18:03 auxym