scipopt / russcip

Rust interface for SCIP
https://crates.io/crates/russcip
Apache License 2.0
37 stars 11 forks source link

Try enabling windows ci workflow #116

Closed mmghannam closed 11 months ago

codecov[bot] commented 11 months ago

Codecov Report

Merging #116 (0370c4a) into main (3e2c6a9) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #116   +/-   ##
=======================================
  Coverage   74.96%   74.96%           
=======================================
  Files          13       13           
  Lines        1725     1725           
=======================================
  Hits         1293     1293           
  Misses        432      432           
KarelPeeters commented 11 months ago

FYI getting this to work is probably as simple as adding $SCIPOPTDIR/lib to PATH.

Something like

$Env:PATH += ";$Env:SCIPOPTDIR/lib"

right before cargo build should do the trick.

mmghannam commented 11 months ago

Thanks for the suggestion @KarelPeeters, it's nice to have the support of a windows user 😄

KarelPeeters commented 11 months ago

I resent the accusation that I'm a Windows user, I'd like to think of myself as more of a multi platform user! 🙂

My bad, it should have been /bin (which is where the .dll files are) instead of /lib (which is where the .lib files are, for static linking). Static linking could be something useful to support in the future, but for now to get CI working dynamic linking is enough.

mmghannam commented 11 months ago

It works now, thanks @KarelPeeters for saving the windows CI 😄

KarelPeeters commented 11 months ago

Nice!