Rosettea / Hilbish

🌓 The Moon-powered shell! A comfy and extensible shell for Lua fans! 🌺 ✨
https://rosettea.github.io/Hilbish/
MIT License
496 stars 21 forks source link

Shell locked on command substitution #261

Closed leath-dub closed 11 months ago

leath-dub commented 11 months ago

Hilbish version: v2.1.2 (Pansy)

I am not sure if this is a bug or if I am missing something, I am new to hilbish. I assume hilbish does not support the "<(command)" syntax from bash and zsh however when you use it there is no error when you attempt to use it.

e.g: try run this

ls <(echo hello)

on bash or zsh this will just print a path but on hilbish you get a path and the shell locks up (ctrl-d and ctrl-c or any keystroke does nothing)

Edit: after some more investigation, seems like the following works

$ echo hello | paste - <(echo hello)
hello    hello

Which suggests its an issue of not closing the temp file if nobody reads from it ? I haven't looked into the source code so I am just guessing

TorchedSammy commented 11 months ago

can you try running on the master branch? otherwise i'll see if updating the sh interpreter package fixes it

leath-dub commented 11 months ago

can you try running on the master branch? otherwise i'll see if updating the sh interpreter package fixes it

same issue on master

TorchedSammy commented 11 months ago

can you report the issue to upstream? i'm sure you'll find the issue is also present on their latest commit by go run cmd/gosh/main.go

leath-dub commented 11 months ago

can you report the issue to upstream? i'm sure you'll find the issue is also present on their latest commit by go run cmd/gosh/main.go

Yep it is an issue there. Ill make an issue there.