-
Notifications
You must be signed in to change notification settings - Fork 192
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
Ubuntu 20.04 #98
Comments
Good idea! I'll consider that for Rust 1.47.0. I also want to update to PostgreSQL 12 at the same time. |
+1, it's not exactly a big deal, but newer versions of included deps is always nice. e.g. git in the current image doesn't support |
I tried to update to Ubuntu 20.04 for the Rust 1.49.0 release, but it broke the |
This might be worth revisiting at some point, now that I'm getting CI working on GitHub. This will make it easier to run all the linking tests on PRs, to see if they're working. |
I have just tried Ubuntu 20.04 again, and
At least one major hosted PostgreSQL provider, Citus Data, uses certs that break |
The issue with the static link check is slightly different, as I have described in my comment of the linked PR. The gist is: the check is not reliable and I went a different route to test this. |
@asaaki Ah, thank you! I hadn't seen that note. I'm going to copy the affected binaries back to main container, and check the linking there. And I'll aim to release Ubuntu 20.04 support when Rust 1.58 ships, to avoid upgrading the existing 1.57 version of the image. Thank you so much for taking the time to figure out what was going wrong! |
Many thanks to @asaaki for having figured out the static link check was returning flase negatives with Alpine's `lld`. See: #120 (comment) The plan is to merge this when Rust 1.58.0 is released, so people who need 18.04 can stick with1.57.0. Closes #98. Closes #120.
As the title says. Would be nice to have access to newer packages when building customized CI images.
The text was updated successfully, but these errors were encountered: