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 inim@0.6.1
    Reading official package list
   Checking for cligen@>= 1.2.0
      Info: Dependency on cligen@>= 1.2.0 already satisfied
  Verifying dependencies for cligen@1.2.2
    Reading official package list
   Checking for noise@>= 0.1.4
      Info: Dependency on noise@>= 0.1.4 already satisfied
  Verifying dependencies for noise@0.1.4
 Installing inim@0.6.1
   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 inim@0.6.1
      Info: Dependency on cligen@>= 1.2.0 already satisfied
  Verifying dependencies for cligen@1.5.23
      Info: Dependency on noise@>= 0.1.4 already satisfied
  Verifying dependencies for noise@0.1.4
 Installing inim@0.6.1
   Building inim/inim using c backend
    Prompt: inim@0.6.1 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 cligen@1.3.2
Downloading https://github.com/c-blake/cligen.git using git
  Verifying dependencies for cligen@1.3.2
 Installing cligen@1.3.2
   Success: cligen installed successfully.

~ took 4s
❯ nimble install inim
Downloading https://github.com/AndreiRegiani/INim using git
  Verifying dependencies for inim@0.6.1
      Info: Dependency on cligen@>= 1.2.0 already satisfied
  Verifying dependencies for cligen@1.3.2
      Info: Dependency on noise@>= 0.1.4 already satisfied
  Verifying dependencies for noise@0.1.4
 Installing inim@0.6.1
   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 'inim@0.6.1', 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

cligen@1.3.2 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