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

Add build-essential apt package to builder image #8

Merged
merged 1 commit into from
Jul 23, 2023

Conversation

orangejulius
Copy link
Member

@orangejulius orangejulius commented Jul 23, 2023

As in pelias/docker-libpostal_baseimage#11, Ubuntu 22 has some slightly differing package install semantics where we need to install the build-essential package to get make and all the other tools needed to compile against libpostal.

This Dockerfile uses a builder image to do all the compiling, and then the compiled results are moved to a new, slim, Docker image, so this change doesn't affect the final image much.

It's just needed to keep everything working for Ubuntu 22

pelias/pelias#951

As in pelias/docker-libpostal_baseimage#11,
Ubuntu 22 has some slightly differing package install semantics where we
need to install the `build-essential` package to get `make` and all the
other tools needed to compile against libpostal.

This Dockerfile uses a builder image to do all the compiling, and then
the compiled results are moved to a new, slim, Docker image, so this
change doesn't affect the final image much.

It's just needed to keep everything working for Ubuntu 22

pelias/pelias#951
@orangejulius orangejulius merged commit ab29b91 into master Jul 23, 2023
1 check passed
@orangejulius orangejulius deleted the ubuntu-22-build branch July 23, 2023 13:54
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.

1 participant