-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Bump docker images to z3 4.8.12 #11672
Conversation
|
|
|
|
|
|
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.
Looks fine.
@@ -37,7 +37,7 @@ RUN set -ex; \ | |||
cmake ninja-build \ | |||
libboost-filesystem-dev libboost-test-dev libboost-system-dev \ | |||
libboost-program-options-dev \ | |||
libcvc4-dev libz3-static-dev \ | |||
libcvc4-dev libz3-static-dev z3-static \ | |||
; \ | |||
apt-get install -qy python3-pip python3-sphinx; \ |
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.
If another rebuild happen to be needed for any reason, could you also remove Sphinx from here?
apt-get install -qy python3-pip python3-sphinx; \ | |
apt-get install -qy python3-pip; \ |
Related to #11662 (comment). Our docs.sh
script reinstalls Sphinx anyway. This installs an ancient version (1.8.5, latest is 4.1.0) which actually causes problems because pip
does not by default update packages if they're already installed. I wanted to change it myself but my PR fixes the problem in a different way so it's not strictly required. Still, would be nice not to have this package here any more.
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.
Just this one?
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.
Yeah. python3-pip
is still needed. It's just Sphinx that's superfluous.
Since we're at it, I can just push that one again. |
b0d2288
to
beab65b
Compare
|
Thanks! |
|
|
|
beab65b
to
f745604
Compare
@chriseth can you merge this? |
After these are built, their hashes will be used in the PR that updates the SMTChecker tests. When those tests pass, we can merge this.