search

DanielGavin / ols

Language server for Odin
MIT License
439 stars 67 forks source link

Build failure on Windows #400

Closed naomipappe closed 4 months ago

naomipappe commented 4 months ago

Cloning and executing Windows build script, I get

$ ./build.bat
E:/ols/src/odin/printer/visit.odin(292:45) Error: Cannot assign value 'path' of type 'string' to '^Expr' in procedure argument 
        document = cons(document, visit_expr(p, path)) 
                                                ^~~^
E:/ols/src/odin/printer/visit.odin(299:54) Error: Cannot assign value 'v.fullpaths[0]' of type 'string' to '^Expr' in procedure argument
        ... expr(p, v.fullpaths[0])) ...

This is reproduced on commit b3ed829bd3623748a34b9031e27e8c693a226232 on master branch.

DanielGavin commented 4 months ago

You need a later version. There was a change in core:odin/ast recently.

naomipappe commented 4 months ago

Thanks! I just started with Odin and forgot that it is actively worked on.

Thank you for your work on the language server!