search

hans-ekbrand / lc0

The rewritten engine, originally for tensorflow. Now all other backends have been ported here.
GNU General Public License v3.0
3 stars 1 forks source link

Appveyor.yml support for recent version of Boost #6

Closed Mr-Twave closed 2 years ago

Mr-Twave commented 2 years ago

This is a change that is analogous to the one made in the killerducky version of lc0.

hans-ekbrand commented 2 years ago

The problem is already solved, see https://github.com/hans-ekbrand/lc0/blob/improved_integration/appveyor.yml#L25

hans-ekbrand commented 2 years ago

Oh, I see now that your PR was against the old branch a/b-helper. I don't actually develop that branch anymore, all current work is in the branch improved_integration (which already has a fix for the problem).

Your PR does not build cleanly on appveyor, probably due to the boost version (see the fix, which has the correct boost version). Feel free to update the boost version, and if it then builds cleanly, I'll accept it.

hans-ekbrand commented 2 years ago

Appveyor works fine with the current branch.