Skip to content

Comments

[Backport 2.30-maintenance] libexpr: Don't use nix::dirOf in prim_dirOf (fix 2.23 regression)#14522

Merged
internal-nix-ci[bot] merged 2 commits into2.30-maintenancefrom
backport-14515-to-2.30-maintenance
Nov 9, 2025
Merged

[Backport 2.30-maintenance] libexpr: Don't use nix::dirOf in prim_dirOf (fix 2.23 regression)#14522
internal-nix-ci[bot] merged 2 commits into2.30-maintenancefrom
backport-14515-to-2.30-maintenance

Conversation

@internal-nix-ci
Copy link

Automatic backport to 2.30-maintenance, triggered by a label in #14515.

These will change in the next commit to fix the silent regression from 2.23
in the handling of multiple subsequent path separators.

(cherry picked from commit 86f0908)
This gets us back to pre-2.23 behavior of this primop.
Done by inlining the code of `nix::dirOf` from 2.2-maintenance.

(cherry picked from commit a33fccf)
@internal-nix-ci internal-nix-ci bot requested a review from edolstra as a code owner November 9, 2025 17:43
@internal-nix-ci internal-nix-ci bot enabled auto-merge November 9, 2025 17:43
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Nov 9, 2025
@internal-nix-ci internal-nix-ci bot merged commit 95fbf92 into 2.30-maintenance Nov 9, 2025
15 checks passed
@internal-nix-ci internal-nix-ci bot deleted the backport-14515-to-2.30-maintenance branch November 9, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant