Skip to content
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.

Sign up for GitHub

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

Jump to bottom

od: Avoid subshell in rcS and rcK #104

Draft
wants to merge 1 commit into
base: opendingux
Choose a base branch
from
Draft

od: Avoid subshell in rcS and rcK #104

wants to merge 1 commit into from

Commits on Dec 16, 2022

  1. od: Avoid subshell in rcS and rcK 

    We previously used `( ... )` subprocess for each init script
to prevent the `exit` command from exiting `rcS/K`.

Instead, define the `EXIT=return` in `rcS/K` and use `${EXIT:-exit}`
in the scripts instead of the `exit` command.

`${EXIT:-exit}` will expand to `exit` if `$EXIT` is not set,
allowing these scripts to be called directly as well.

This only applies to system scripts. We still use subshells
in `S99localscripts.sh` to avoid breaking any user scripts.

Signed-off-by: Gleb Mazovetskiy <[email protected]>
    @glebm
    glebm committed Dec 16, 2022
    Configuration menu
    Copy the full SHA
    24e8cff View commit details
    Browse the repository at this point in the history