diff options
author | Griffin Smith <grfn@gws.fyi> | 2020-08-27T15·58-0400 |
---|---|---|
committer | glittershark <grfn@gws.fyi> | 2020-08-28T01·18+0000 |
commit | fb0528c5dada2d3af6152b37a46e505e6bb2ac94 (patch) | |
tree | fa921ed14ef82642d1454ef5adff1f56fdf00037 /users/glittershark/system/pkgs/clang-tools.nix | |
parent | 3bada1d41eed3bd8c72d6811147cd100fcb585ca (diff) |
fix(tvix): Pass all args when auto-calling a function with an ellipsis r/1727
The command line options --arg and --argstr that are used by a bunch of CLI commands to pass arguments to top-level functions in files go through the same code-path as auto-calling top-level functions with their default arguments - this, however, was only passing the arguments that were *explicitly* mentioned in the formals of the function - in the case of an as-pattern with an ellipsis (eg args @ { ... }) extra passed arguments would get omitted. This fixes that to instead pass *all* specified auto args in the case that our function has an ellipsis. Submitted upstream at https://github.com/NixOS/nix/pull/3965 Fixes: #46 Change-Id: I32b7ee0e5bacf75b2bc43a3f0796f533f4bd5959 Reviewed-on: https://cl.tvl.fyi/c/depot/+/1863 Reviewed-by: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
Diffstat (limited to 'users/glittershark/system/pkgs/clang-tools.nix')
0 files changed, 0 insertions, 0 deletions