bash-preexec about bash-preexec HOT 4 CLOSED

Cool man, any particular reason for the fork? I'm interested in trying to address use cases that my current implementation may not support. Not to mention bugs :) Thanks for your interest!

from bash-preexec.

Ja, there were two reasons:

• I have removed the sed scripts and (the very smart!) invocation/use of __bp_trim_whitespace() in __bp_precmd_invoke_cmd() to speed up the thing a bit
• access to $? in the precmd mechanism. Currently, I am using it like this

In my last commit, preexecution is beeing allowed, even though history has been disabled

from bash-preexec.

Yeah the use of sed to save the previous prompt command and the trimming is all stuff that is a little heavy. I actually just needed to get access to $? as well so I wrote 417c01b to sort of simulate it.

Thanks for your feedback and input. Feel free to open anything or put in a pull request if you want to make changes.

from bash-preexec.

Thanks!

from bash-preexec.