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

Use bionic and update dependencies #67

Closed
wants to merge 1 commit into from
Closed

Conversation

xrl
Copy link

@xrl xrl commented Jan 10, 2019

I had trouble building my CLI app with diesel/openssl support. Turns out it does matter the order of extern crate foo; does matter. While I threw everything possible at the wall, I tried building my own rust-musl-builder image.

One of the g++ dependency was failing to install, owing to something wrong in the Ubuntu archives no doubt. And I use bionic elsewhere in my infrastructure so I thought it would be helpful to upgrade to a new LTS release. It was pretty easy, the biggest change is dragging in gcc8 libraries vs 4.7.

I also upgrade the custom built dependencies. OpenSSL gets LTS release 1.0.2q, postgres gets bumped to 9.6.9.

I don't think the linked issue rust-lang/rust#34978 matters anymore. I have successfully built using this image and I'm happy with it.

Also, pro-tip to anyone who wants to build their own rust-musl for nightly, use docker build's build-args to override the toolchain:

docker build --build-arg="TOOLCHAIN=nightly-2019-01-09" -t rust-musl:nightly .

@emk
Copy link
Owner

emk commented Mar 4, 2019

Thank you for this PR! You're right, we should upgrade this to the latest LTS.

@emk
Copy link
Owner

emk commented Apr 19, 2019

I've upgraded to the latest LTS, OpenSSL, etc. You can find it on the experimental branch while I test it against various existing apps. Thank you!

@emk emk closed this Apr 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants