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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Do not change global env #675

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Do not change global env #675

wants to merge 3 commits into from

Commits on Dec 27, 2023

  1. Do not change ENV variables of a current process 

    Instead pass relevant environment to the actual shell command being run

Signed-off-by: James Couball <jcouball@yahoo.com>
    @jcouball
    jcouball committed Dec 27, 2023
    Configuration menu
    Copy the full SHA
    4a969a0 View commit details
    Browse the repository at this point in the history

  2. Test using Open3.popen2 instead of IO.popen 

    Signed-off-by: James Couball <jcouball@yahoo.com>
    @jcouball
    jcouball committed Dec 27, 2023
    Configuration menu
    Copy the full SHA
    4231e94 View commit details
    Browse the repository at this point in the history

  3. Use Open3.capture3 instead of IO.popen 

    Signed-off-by: James Couball <jcouball@yahoo.com>
    @jcouball
    jcouball committed Dec 27, 2023
    Configuration menu
    Copy the full SHA
    d684854 View commit details
    Browse the repository at this point in the history