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 lib dir to Libdl.DL_LOAD_PATH #64

Closed
wants to merge 7 commits into from
Closed

add lib dir to Libdl.DL_LOAD_PATH #64

wants to merge 7 commits into from

Conversation

larssp
Copy link

@larssp larssp commented Sep 6, 2018

resolves build failure #63

@larssp
Copy link
Author

larssp commented Sep 6, 2018

The MathOptInterface v0.5 deprecation of cantransform, canaddconstraint, canaddvariable, canset, canmodify, and candelete functions has to be resolved next

@@ -1,7 +1,7 @@
julia 0.6
BinDeps
MathProgBase 0.5.5 0.8
MathOptInterface 0.4.1 0.5
MathOptInterface 0.4.1 0.6
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mlubin
Copy link
Member

mlubin commented Sep 6, 2018

Before you go too far down this path, please put changes for updating from MOI 0.5 to 0.6 in a separate PR to keep things easy to review. Also keep in mind that tests need to pass on both Julia 0.6 and 0.7 before we merge. If you can get them passing on 1.0 as well, that's even better.

@@ -2,7 +2,7 @@ module CbcMathProgSolverInterface

using Cbc.CbcCInterface

importall MathProgBase.SolverInterface
using MathProgBase.SolverInterface

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

using only replaces parts of what importall was doing, but see #65.

@larssp
Copy link
Author

larssp commented Sep 9, 2018

this PR is premature, closing it.

@larssp larssp closed this Sep 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants