postmodern / chruby

Changes the current Ruby
MIT License
2.87k stars 190 forks source link

Make chruby-exec exec #384

Closed egh closed 7 years ago

egh commented 7 years ago

Previous, chruby exec would leave a parent bash process around. Now it will exec the ruby command, leaving no parent bash process.

This is important when, for instance, starting ruby processes from upstart. Without this fix, upstart will be tracking the parent bash process, not the child ruby process.

It is also, in my opinion, the correct thing to do. With this change, chruby-exec starts your ruby process and gets out of the way. Without this fix, you always have an old bash process hanging around.

postmodern commented 7 years ago

Good points.

egh commented 7 years ago

Thank you!