search

arnetheduck / nph

An opinionated code formatter for Nim
Other
77 stars 12 forks source link

Build failing when following installation instructions #2

Closed auxym closed 6 months ago

auxym commented 6 months ago

The instructions state:

nimble -l setup
nimble build

nimble build fails when I attempt this:

❯ cd nph/

nph on ξ‚  master via πŸ‘‘ v2.0.0 
❯ nimble -l setup
   Warning: Using project local deps mode
  Verifying dependencies for nph@0.1
     Info:  "nimble.paths" is updated.
     Info:  "config.nims" is already set up.

nph on ξ‚  master [!] via πŸ‘‘ v2.0.0 
❯ nimble build
   Warning: Using project local deps mode
  Verifying dependencies for nph@0.1
   Building nph/nph using c backend
/home/francis/source/nph/src/astcmp.nim(6, 8) Error: cannot open file: ast
       Tip: 2 messages have been suppressed, use --verbose to show them.
nimble.nim(229)          buildFromDir

    Error:  Build failed for the package: nph

Nim and nimble versions:

nph on ξ‚  master [!] via πŸ‘‘ v2.0.0 
❯ nimble --version
   Warning: Using project local deps mode
nimble v0.14.2 compiled at 2023-08-01 01:21:33
git hash: couldn't determine git hash

nph on ξ‚  master [!] via πŸ‘‘ v2.0.0 
❯ nim --version
Nim Compiler Version 2.0.0 [Linux: amd64]
Compiled at 2023-08-01
Copyright (c) 2006-2023 by Andreas Rumpf

git hash: a488067a4130f029000be4550a0fb1b39e0e9e7c
active boot switches: -d:release
arnetheduck commented 6 months ago

5 maybe, reopen if not ;)

auxym commented 6 months ago

Should have mentioned, I am on linux (manjaro).

I pulled and now get the following:

nph on ξ‚  master via πŸ‘‘ v2.0.0 
❯ nimble -l setup
   Warning: Using project local deps mode
  Verifying dependencies for nph@0.1
     Info:  "nimble.paths" is updated.
     Info:  "config.nims" is already set up.

nph on ξ‚  master [!] via πŸ‘‘ v2.0.0 
❯ nimble build
   Warning: Using project local deps mode
  Verifying dependencies for nph@0.1
   Building nph/nph using c backend
/home/francis/source/nph/src/phast.nim(15, 32) Error: cannot open file: idents
       Tip: 2 messages have been suppressed, use --verbose to show them.
nimble.nim(229)          buildFromDir

    Error:  Build failed for the package: nph
arnetheduck commented 6 months ago

can you give https://github.com/arnetheduck/nph/pull/6 a shot? not sure what's wrong with your setup, works for both me and CI

auxym commented 6 months ago

Just pulled master (68629bc72), works now, I'm closing. Thank you!