From c3c52231f02e9ea1cb5a77128734719c1738d845 Mon Sep 17 00:00:00 2001 From: Wolfgang Walther Date: Wed, 6 Mar 2024 19:07:41 +0100 Subject: [PATCH] postgresql: remove left-over LC_ALL environment variable This was introduced in 65fd8f3f48fed5e2a4b82da7e354fac53ae72099 around 20 years ago as LANG=en_US, later changed to LC_ALL=en_US and LC_ALL=C to avoid bash warnings. This might have been relevant to run initdb during the check phase, but the PostgreSQL docs [1] say this today: For implementation reasons, setting LC_ALL does not work for this purpose Therefore it can be removed safely. [1]: https://www.postgresql.org/docs/current/regress-run.html#REGRESS-RUN-LOCALE --- pkgs/servers/sql/postgresql/generic.nix | 2 -- 1 file changed, 2 deletions(-) diff --git a/pkgs/servers/sql/postgresql/generic.nix b/pkgs/servers/sql/postgresql/generic.nix index e1781352bbc5..2608bfb0207e 100644 --- a/pkgs/servers/sql/postgresql/generic.nix +++ b/pkgs/servers/sql/postgresql/generic.nix @@ -131,8 +131,6 @@ let installTargets = [ "install-world" ]; - LC_ALL = "C"; - postPatch = '' # Hardcode the path to pgxs so pg_config returns the path in $out substituteInPlace "src/common/config_info.c" --subst-var out