mosdepth icon indicating copy to clipboard operation
mosdepth copied to clipboard

Build failure with nim 2.0.0

Open meanjollies opened this issue 1 year ago • 3 comments

Hello, when I attempt to build the latest release, I get the following:

$ nim c -d:danger mosdepth.nim
Hint: used config file '/etc/nim/nim.cfg' [Conf]
Hint: used config file '/etc/nim/config.nims' [Conf]
Hint: used config file '/home/a/mosdepth/mosdepth-0.3.5/nim.cfg' [Conf]
......................................................................................................................................................................
/home/a/mosdepth/mosdepth-0.3.5/mosdepth.nim(261, 18) template/generic instantiation of `tag` from here
/home/a/.nimble/pkgs2/hts-0.3.25-c05c9adf6b2e10a7f07edfef29110ea14bb66611/hts/bam/auxtags.nim(63, 7) Warning: unreachable code after 'return' statement or '{.noReturn.}' proc [UnreachableCode]
/home/a/mosdepth/mosdepth-0.3.5/mosdepth.nim(261, 18) template/generic instantiation of `tag` from here
/home/a/.nimble/pkgs2/hts-0.3.25-c05c9adf6b2e10a7f07edfef29110ea14bb66611/hts/bam/auxtags.nim(66, 7) Warning: unreachable code after 'return' statement or '{.noReturn.}' proc [UnreachableCode]
/home/a/mosdepth/mosdepth-0.3.5/mosdepth.nim(261, 18) template/generic instantiation of `tag` from here
/home/a/.nimble/pkgs2/hts-0.3.25-c05c9adf6b2e10a7f07edfef29110ea14bb66611/hts/bam/auxtags.nim(74, 7) Warning: unreachable code after 'return' statement or '{.noReturn.}' proc [UnreachableCode]
/home/a/mosdepth/mosdepth-0.3.5/mosdepth.nim(504, 3) Error: undeclared identifier: 'shallow'
candidates (edit distance, scope distance); see '--spellSuggest': 
 (3, 4): 'alloc'
 (3, 4): 'dealloc'
 (3, 4): 'realloc'

This began once I upgraded to nim 2.0.0.

meanjollies avatar Sep 16 '23 18:09 meanjollies

Hi, for now, I suggest to use nim < 2.0.0. I'll update to latest nim in the neext few months.

brentp avatar Sep 17 '23 08:09 brentp

Getting the same when trying to update the bioconda recipe at https://dev.azure.com/bioconda/bioconda-recipes/_build/results?buildId=43039&view=logs&j=1b052f1d-4456-52f0-9d43-71c4c5bd734d&t=edc48dcd-1fc2-5e3b-7036-7be9cea00123&l=376

I'll change the nim in meta.yaml to nim < 2.0.0 (see PR at https://github.com/bioconda/bioconda-recipes/pull/43500).

pdiakumis avatar Oct 06 '23 01:10 pdiakumis

Is there any progress on the update to Nim 2.x ? I am trying to add a linux-aarch64 Bioconda package for mosdepth at https://github.com/bioconda/bioconda-recipes/pull/47052 but https://anaconda.org/conda-forge/nim provides on nim 2.0.4 for linux-aarch64...

martin-g avatar Apr 10 '24 07:04 martin-g