emersion / mrsh

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

Job control improvements #175

Closed caseman closed 3 years ago

caseman commented 3 years ago

Draft fix for https://github.com/emersion/mrsh/issues/174

I included a jobs.sh test script, but it isn't incorporated into the test target because it causes it hang. Running it separately it works fine though and exits cleanly. Also not sure if these tests are only for parity, if so then it needs more attention since I didn't attempt to run it in other shells yet.

See also XXX comments inline. In particular the one about the general fork abstraction. I feel like forking a job process could be generalized since it happens in several places.

caseman commented 3 years ago

Superceded by: https://github.com/emersion/mrsh/pull/176