Add builtins.canonicalPath · NixOS/nix#13111

(5 comments) (1 reaction) (0 assignees)C++ (9,803 stars) (1,293 forks)batch import

featuregood first issueidea approved

説明

Is your feature request related to a problem?

Symlinks in nixpkgs lib.fileset are causing issues: https://github.com/NixOS/nixpkgs/issues/393845 and this function would help resolve it.

Proposed solution

Add a a builtins.canonicalPath with the same behavior as the realpath command, as suggested by @infinisil .

Alternative solutions

Additional context

Possibly somewhat related: https://github.com/NixOS/nix/issues/2109 (but this feature request would only expose a new nix function, not change anything else inside the nixcpp code itself.

Checklist

checked latest Nix manual (source)
checked open feature issues and pull requests for possible duplicates

Add :+1: to issues you find important.

コントリビューターガイド

技術スタック: cpp
領域: backendtooling
Issue 種別: feature
難度: 5
推定時間: over 1 week
活動状況: active
明確さ: clear
前提条件: Familiarity with C++Understanding of Nix evaluator internalsKnowledge of symlink handling
初心者向け度: 10
調査方針: Study the implementation of existing built in functions like `builtins.readFile` or `builtins.toString` in the Nix source code, particularly in `src/libexpr/primops.cc`. Review the discussion in the linked issue #393845 about symlink problems in `lib.fileset`. Look at the `realpath` command behavior and how to expose it as a built in. Consider the related issue #2109 for context. No linked PRs exist yet, so focus on understanding the evaluator's primitive registration mechanism.