Skip to content

fix: use POSIX compliant [ ] syntax instead of bash-only [[ ]] in env --use-on-cd - #1595

Closed
RohithPariki wants to merge 1 commit into
Schniz:masterfrom
RohithPariki:fix/issue-1591-posix-compliant-bash-env
Closed

RohithPariki wants to merge 1 commit into
Schniz:masterfrom
RohithPariki:fix/issue-1591-posix-compliant-bash-env

Conversation

@RohithPariki

@RohithPariki RohithPariki commented Sep 12, 2026 •

Copy link
Copy Markdown

Fixes #1591 by using the POSIX compliant [ ]\ syntax instead of the bash-only [[ ]]\ in the script generated by \ nm env --use-on-cd\ for bash and zsh. This ensures compatibility for users whose /bin/sh\ is \dash\ and who eval the output of \ nm env\ in global profile scripts.

@RohithPariki

Copy link
Copy Markdown
Author

Closing PR: This was an automated experimental submission while testing a new contribution workflow, and upon review, the implementation is too trivial to justify maintainer review time. Apologies for the noise!

@RohithPariki

Copy link
Copy Markdown
Author

Closing PR: Re-evaluated and this is too trivial of a change. Closing permanently.

@RohithPariki
RohithPariki deleted the fix/issue-1591-posix-compliant-bash-env branch September 12, 2026 20:16
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.

fnm env --use-on-cd emits bash-only [[ ]] syntax with no POSIX sh option

1 participant