Skip to content
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

Doesn't compile with 1.6.0 on macOS #123

Open
ynfle opened this issue Oct 19, 2021 · 3 comments
Open

Doesn't compile with 1.6.0 on macOS #123

ynfle opened this issue Oct 19, 2021 · 3 comments
Labels
question Further information is requested

Comments

@ynfle
Copy link

ynfle commented Oct 19, 2021

 $ 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
@auxym
Copy link
Contributor

auxym commented Dec 16, 2021

Same error on Windows 10, Nim 1.6.0.

@0atman
Copy link
Collaborator

0atman commented Mar 30, 2022

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 0atman added the question Further information is requested label Mar 30, 2022
@auxym
Copy link
Contributor

auxym commented Mar 30, 2022

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants