Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Windows cross compile #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NickHu
Copy link

@NickHu NickHu commented Jul 13, 2023

A continuation of the work started in #28.

This is essentially bf9af72 but with support for https://github.com/cross-rs/cross.

Testing

  1. Install cross
    1a. Install the windows MSVC target as described here: https://github.com/cross-rs/cross-toolchains
  2. Download and extract a release from here (e.g. https://github.com/coin-or/Cbc/releases/download/releases%2F2.10.10/Cbc-releases.2.10.10-w64-msvc17-md.zip)
  3. CBC_ROOT=<path to extracted Cbc library> cross build --target=x86_64-pc-windows-msvc --example knapsack

@NickHu
Copy link
Author

NickHu commented Jul 13, 2023

I think #2 is obsolete with respect to this.

@Thell could you please comment on why 9e064a6 exists in #28? It seems to just break things for me.

@Thell
Copy link

Thell commented Jul 13, 2023

I think #2 is obsolete with respect to this.

@Thell could you please comment on why 9e064a6 exists in #28? It seems to just break things for me.

I believe that the reason it breaks things for you is that you are using a pre-built release package which, if I remember correctly, does not support parallel and has a libXX naming convention -vs- building from source.
Don't quote me on that since I had quite a adventure trying to get a thread safe windows build up and running on Windows. Even after using Google's prebuilt parallel release I found CBC too slow for what I was working on and ended up using HiGHS which reduced the runtime of my test sets from ~57minutes to ~13minutes and my actual full run from months to days (not to say HiGHS wasn't a problem filled adventure of its own, because it was).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants