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 Xenial as the build image's base distribution #84

Merged
merged 4 commits into from
May 11, 2020

Conversation

jryans
Copy link
Contributor

@jryans jryans commented May 7, 2020

Since Xenial has the oldest glibc (2.23) of all the active distributions, this
attempts to construct a build image based on that. The Dockerfile here was built
by recursively expanding the FROM lines until reaching
buildpack-deps:bionic-curl and then changing that to xenial. Everything else
is the same.

@cknoll has tested a nightly build prepared this way, and verified it fixes the segfault.

Part of element-hq/element-web#13553

jryans added 3 commits May 7, 2020 14:45
Since Xenial has the oldest glibc (2.23) of all the active distributions, this
attempts to construct a build image based on that. The Dockerfile here was built
by recursively expanding the `FROM` lines until reaching
`buildpack-deps:bionic-curl` and then changing that to `xenial`. Everything else
is the same.

Part of element-hq/element-web#13553
@jryans jryans requested a review from a team May 7, 2020 15:48
@dbkr
Copy link
Member

dbkr commented May 7, 2020

We should probably merge the apt-get libsqlcipher into the other apt-get so we only update & purge once. Also looks like the linter is broken?

@jryans
Copy link
Contributor Author

jryans commented May 7, 2020

Also looks like the linter is broken?

Yeah, filed element-hq/element-web#13584 on this.

@jryans jryans requested review from dbkr and removed request for a team May 7, 2020 16:47
@jryans jryans merged commit 18f4231 into develop May 11, 2020
@t3chguy t3chguy deleted the jryans/docker-xenial branch May 3, 2022 09:58
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