Skip to content

Update to 17.5, 16.9, 15.13, 14.18, 13.21 #1340

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

Merged
merged 2 commits into from
May 8, 2025

Conversation

tianon
Copy link
Member

@tianon tianon commented May 8, 2025

The automated update keeps failing build tests for various (flaky build type) reasons, so I'm opening this to explicitly race the retries and see if we can get to success or more clear failures (especially ones that aren't the ongoing issues we've been seeing for days on gnu.org).

@tianon
Copy link
Member Author

tianon commented May 8, 2025

Hmm, I wonder if updating config.guess/config.sub even matters anymore for any of the architectures we support. 🤔

The copies in 13.21 were last updated 5 years ago (https://github.com/postgres/postgres/tree/REL_13_21/config) but in 17.5 they're barely a year old (https://github.com/postgres/postgres/tree/REL_17_5/config).

We could do some build tests to find out, or we could just remove them and find out downstream. 🙈

(I think riscv64 is the only one I'd maybe worry about at this point, but even 13.21 seems to have support for it, so we're probably totally fine and could stop doing this.)

@tianon
Copy link
Member Author

tianon commented May 8, 2025

Oh, we're pulling those from a specific commit, and it's a commit that's older than the file in 13.21 (2017 vs 2020), so we're already doing net worse than upstream and we should absolutely remove these lines! ✨

When we added these lines, we needed newer files than PostgreSQL had upstream, but the situation has changed and we were pinned to a specific commit from 2017 - the _oldest_ PostgreSQL release version has a file from 2020.
@tianon tianon merged commit 437897c into docker-library:master May 8, 2025
32 checks passed
@tianon tianon deleted the update branch May 8, 2025 18:55
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request May 8, 2025
Changes:

- docker-library/postgres@437897c: Merge pull request docker-library/postgres#1340 from infosiftr/update
- docker-library/postgres@b234702: Remove explicit config.guess/config.sub updates
- docker-library/postgres@2667482: Update to 17.5, 16.9, 15.13, 14.18, 13.21
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