-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
or-tools: 9.4 -> 9.7 #272425
or-tools: 9.4 -> 9.7 #272425
Conversation
81b51bf
to
1ac6b73
Compare
@ofborg build or-tools or-tools.passthru.tests |
@ofborg eval |
pybind11_protobuf = fetchFromGitHub { | ||
owner = "pybind"; | ||
repo = "pybind11_protobuf"; | ||
rev = "b713501f1da56d9b76c42f89efd00b97c26c9eac"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to https://github.com/google/or-tools/blob/v9.7/WORKSPACE#L228, this should be 5baa2dc9d93e3b608cde86dfa4b8c63aeab4ac78
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to use b713501f1da56d9b76c42f89efd00b97c26c9eac
for the build to succeed, probably due to some mismatches as in google/or-tools#4027.
This should also drop |
Reopening the PR as or-tools is failing on master. I'll do the next bump to 9.9 in other PR, whose diff is way more significant. |
Description of changes
Not to 9.8 as it requires absl-py 2.0 and #264902.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.