Skip to content

fix(wrapperModules.nushell): make it work in mac shebangs#296

Merged
BirdeeHub merged 1 commit intomainfrom
nu
Feb 11, 2026
Merged

fix(wrapperModules.nushell): make it work in mac shebangs#296
BirdeeHub merged 1 commit intomainfrom
nu

Conversation

@BirdeeHub
Copy link
Owner

@BirdeeHub BirdeeHub commented Feb 11, 2026

wrapperImplementation = "binary" to use pkgs.makeBinaryWrapper as the backend.

This is not really for speed, it is about the same in terms of speed as they are short wrappers that deal mostly with literal strings and then transfer control to another process.

You also get less control over how stuff is escaped in the final result, as the binary cannot run shell code, and any variable expansion allowed (by default not possible as it uses lib.escapeShellArg by default) will happen at build time

However on mac shebangs need to be binaries. This is important for shells and other things which are used in shebangs.

`wrapperImplementation = "binary"` to use `pkgs.makeBinaryWrapper` as
the backend.

This is not really for speed, it is about the same in terms of speed,
however on mac shebangs need to be binaries.
@BirdeeHub BirdeeHub merged commit b0156b2 into main Feb 11, 2026
2 checks passed
@BirdeeHub BirdeeHub deleted the nu branch February 11, 2026 02:12
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.

1 participant