Fix temporary IFS settings - Githubissues

wfxr / forgit

:zzz: A utility tool powered by fzf for using git interactively.

MIT License

4.32k stars 136 forks source link

Fix temporary IFS settings #297

Closed carlfriedrich closed 1 year ago

carlfriedrich commented 1 year ago

When setting the IFS for a certain command only, the command must not be an assignment, otherwise both assignments are evaluated permanently (see https://unix.stackexchange.com/a/458901/317320).

Check list

[x] I have performed a self-review of my code
[ ] I have commented my code in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

Description

Type of change

[x] Bug fix
[ ] New feature
[ ] Refactor
[ ] Breaking change
[ ] Documentation change

Test environment

Shell
- [x] bash
- [ ] zsh
- [ ] fish
OS
- [x] Linux
- [ ] Mac OS X
- [ ] Windows
- [ ] Others:

carlfriedrich commented 1 year ago

This should have been part of #296 already. Getting tired obviously. 🙈 Merging this directly for the same reasons.