Skip to content

Build PostgreSQL with ICU on Ubuntu - #10260

Open
natalya-aksman wants to merge 1 commit into
mainfrom
build_pg_with_icu_on_linux
Open

Build PostgreSQL with ICU on Ubuntu#10260
natalya-aksman wants to merge 1 commit into
mainfrom
build_pg_with_icu_on_linux

Conversation

@natalya-aksman

@natalya-aksman natalya-aksman commented Jul 14, 2026

Copy link
Copy Markdown
Member

Build PostgreSQL with ICU support on Linux platform so that we can test nondeterministic collations.

Disable-check: force-changelog-file

@github-actions

Copy link
Copy Markdown

@melihmutlu, @antekresic: please review this pull request.

Powered by pull-review

@natalya-aksman
natalya-aksman requested review from Poroma-Banerjee, akuzm and svenklemm and removed request for antekresic and melihmutlu July 14, 2026 18:34
@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

cd ~/$PG_SRC_DIR
./configure --prefix=$HOME/$PG_INSTALL_DIR --with-openssl \
--without-readline --without-zlib --without-libxml ${{ matrix.pg_extra_args }}
--without-readline --without-zlib --without-libxml --with-icu ${{ matrix.pg_extra_args }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gh_matrix_builder.py also defines --without-icu, so might be better to move it there, to at least centralize this in one place.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also have --without-icu defined in linux-32bit-build-and-test.yaml.
It may be easier to limit the scope of application of --with-icu to specific place as gh_matrix_builder.py configs get included in many places?

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