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

Downgrade llvm to 15 to fix jit support #1077

Merged
merged 1 commit into from May 12, 2023

Conversation

yosifkit
Copy link
Member

@yosifkit yosifkit commented May 12, 2023

PostgreSQL doesn't support llvm 16 (#1076 (comment)):

Thank you for your comments on #1076 pointing me in a direction for a fix ❤️. Apologies for any stress and work that this caused 🙇‍♂️. With the added test in docker-library/official-images#14646, we shouldn't run into this issue again.

Fixes #1076

@yosifkit
Copy link
Member Author

Hopefully DOCKER_PG_LLVM_DEPS should make it easier for PostgreSQL extension installation on Alpine.

If the LLVM_CONFIG or CLANG variables are necessary/useful to extensions I am willing to move them to ENV as well (or maybe scraping them from pg_config --configure is enough?).

Dockerfile-alpine.template Outdated Show resolved Hide resolved
@ImreSamu
Copy link
Contributor

Hopefully DOCKER_PG_LLVM_DEPS should make it easier for PostgreSQL extension installation on Alpine.

Thank you!
My local docker-postgis docker build test was successful with the new DOCKER_PG_LLVM_DEPS ! ✔️

@tianon tianon merged commit 18c1bef into docker-library:master May 12, 2023
17 checks passed
@tianon tianon deleted the llvm-fix branch May 12, 2023 23:28
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request May 12, 2023
Changes:

- docker-library/postgres@18c1bef: Merge pull request docker-library/postgres#1077 from infosiftr/llvm-fix
- docker-library/postgres@a3b0bb6: Downgrade llvm to 15 to fix jit support
- docker-library/postgres@1c1e4ff: Add the ability for us to manually trigger GitHub tests
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.

In Postgres 12.15 Image, LLVMBuildGEP symbol not found.
3 participants