docker-library / postgres

Docker Official Image packaging for Postgres
http://www.postgresql.org
MIT License
2.16k stars 1.13k forks source link

Downgrade llvm to 15 to fix jit support #1077

Closed yosifkit closed 1 year ago

yosifkit commented 1 year ago

PostgreSQL doesn't support llvm 16 (https://github.com/docker-library/postgres/issues/1076#issuecomment-1545209600):

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 https://github.com/docker-library/official-images/pull/14646, we shouldn't run into this issue again.

Fixes https://github.com/docker-library/postgres/issues/1076

yosifkit commented 1 year ago

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?).

ImreSamu commented 1 year ago

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 ! :heavy_check_mark: