-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reorganize pash_runtime #665
Conversation
OS:ubuntu-20.04 |
OS = Debian 10
|
OS:ubuntu-20.04 |
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
7a20b52
to
3d8fd17
Compare
OS:ubuntu-20.04 |
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
OS:ubuntu-20.04 |
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
OS:ubuntu-20.04 |
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
OS:ubuntu-20.04 |
OS:ubuntu-20.04 |
OS = Debian 10
|
OS:ubuntu-20.04 |
OS = Debian 10
|
This PR significantly cleans up the pash runtime. It is now ready to make it an external library. @mgree we can discuss the preferred interface of such a library at some point. What would be the most useful interface. I am thinking something along the lines of: ## Wraps a SCRIPT with a PRE and POST script while making sure to hide this wrapping from SCRIPT
## by passing it the correct error code and propagating its error code outside while also not resetting `set` state
## while executing the PRE and POST scripts.
wrap PRE_SCRIPT SCRIPT [POST_SCRIPT] If we can think of a couple usecases too we can add a cute README and publicize this library as its own repo. |
No description provided.