nimble icon indicating copy to clipboard operation
nimble copied to clipboard

Enables SAT by default to test against the CI

Open jmgomez opened this issue 10 months ago • 3 comments

jmgomez avatar Apr 10 '24 12:04 jmgomez

Compiling Nimble right now gives this error:

/home/tray/Documents/Projects/nimble/src/nimblepkg/nimblesat.nim(337, 35) Error: attempting to call undeclared routine: 'addUnique'
       Tip: 6 messages have been suppressed, use --verbose to show them.
nimble.nim:229           buildFromDir

    Error:  Build failed for the package: nimble

Is the SAT solver done or are you still working on it? I'd be down to helping out if you need any help :)

xTrayambak avatar Apr 11 '24 15:04 xTrayambak

Compiling Nimble right now gives this error:

/home/tray/Documents/Projects/nimble/src/nimblepkg/nimblesat.nim(337, 35) Error: attempting to call undeclared routine: 'addUnique'
       Tip: 6 messages have been suppressed, use --verbose to show them.
nimble.nim:229           buildFromDir

    Error:  Build failed for the package: nimble

The error is because that function is only in newer std versions. Will take care of it.

Is the SAT solver done or are you still working on it?

Im at the point of making it work with all other Nimble commands and it needs real world testing before making it the default, which probably should be aligned with making local install the default too. The cache also needs more work.

jmgomez avatar Apr 11 '24 15:04 jmgomez

Getting closer. Missing one test from tuninstall that only fails in linux and the lock feature

jmgomez avatar Apr 24 '24 14:04 jmgomez