Skip to content

Fix nix-support/ replacements - #186

Merged
Lassulus merged 1 commit into
Lassulus:mainfrom
vog:fix-nix-support
Aug 26, 2026
Merged

Fix nix-support/ replacements#186
Lassulus merged 1 commit into
Lassulus:mainfrom
vog:fix-nix-support

Conversation

@vog

@vog vog commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

This fixes the issue that for some packages such as postgresql, if wrapper via (...).wrapper and included into nativeBuildInput, e.g. by using it in a nix-shell, the PATH contains the original instead of the wrapped PATH.

@vog
vog force-pushed the fix-nix-support branch from ebbc4c3 to f5349ea Compare August 25, 2026 20:43
@vog

vog commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

Improved PR according to feedback.

@Lassulus

Copy link
Copy Markdown
Owner

needs a nix fmt :)

@vog
vog force-pushed the fix-nix-support branch from f5349ea to c836e6d Compare August 26, 2026 13:38
@vog

vog commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

Fixed. It seems that other corners of the code were not properly "nixfmt"'ed as well.

@vog

vog commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

I wonder why the checks are still failing. The lib/default.nix file is now properly formatted by nixfmt 1.4.0.

@vog
vog force-pushed the fix-nix-support branch from c836e6d to 05ae5fa Compare August 26, 2026 13:56
@Lassulus

Copy link
Copy Markdown
Owner

I updated the flake.lock to have nixfmt 1.4.0. you can rebase and CI should succeed

This fixes the issue that for some packages such as postgresql,
if wrapper via (...).wrapper and included into nativeBuildInput,
e.g. by using it in a nix-shell, the PATH contains the original
instead of the wrapped PATH.

Also, run "nixfmt".
@vog
vog force-pushed the fix-nix-support branch from 05ae5fa to d3c0f6e Compare August 26, 2026 15:11
@vog

vog commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

Rebased.

@Lassulus
Lassulus merged commit 28e67b7 into Lassulus:main Aug 26, 2026
@vog
vog deleted the fix-nix-support branch August 26, 2026 17:19
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