emersion / mrsh

A minimal POSIX shell
MIT License
489 stars 35 forks source link

implement hash builtin, hash paths on simple_task #104

Closed apreiml closed 5 years ago

apreiml commented 5 years ago

The only thing missing is to clear the command path cache when the PATH variable is set.

emersion commented 5 years ago

Previously discussed here: https://lists.sr.ht/~emersion/mrsh-dev/%3CImQHuszPupu9KzRmvbTamKE0YrTD1SIWhBQlMDsH4Qs1bJ_d5HrP8Q8cnQ2wXM3q-j2uJHYM8AaOHLXiJUHKg7dHF9GAL33_A2P3K-00Lpo%3D%40emersion.fr%3E

apreiml commented 5 years ago

ah ok. Could've checked that.

emersion commented 5 years ago

Yeah, sorry about the duplicated work... I'm not totally against hash, I just wonder whether it's worth it at all.