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.

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

Revert environment variables instead of setting them to an empty string #5039

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
update env reftest
rjbou committed Nov 10, 2022
commit acd307e1a172d016b310d30c67585ebb673c8538
24 changes: 12 additions & 12 deletions tests/reftests/env.test
Original file line number Diff line number Diff line change
@@ -15,7 +15,7 @@ Done.
### opam env | grep "NV_VARS" | ';' -> ':'
NV_VARS='${BASEDIR}/OPAM/setenv/doc/nv:${BASEDIR}/OPAM/setenv/share/nv': export NV_VARS:
### opam exec -- opam env --revert | grep "NV_VARS" | ';' -> ':'
NV_VARS='': export NV_VARS:
unset NV_VARS:
### NV_VARS=/another/path
### opam env | grep "NV_VARS" | ';' -> ':'
NV_VARS='${BASEDIR}/OPAM/setenv/doc/nv:${BASEDIR}/OPAM/setenv/share/nv:/another/path': export NV_VARS:
@@ -43,9 +43,9 @@ NV_VARS='hej!!'; export NV_VARS;
### opam exec -- opam env | grep "^NV_VARS|^OPAM_SWITCH_PREFIX|${OPAM}"
OPAM_SWITCH_PREFIX='${BASEDIR}/OPAM/conffile'; export OPAM_SWITCH_PREFIX;
NV_VARS='hej!!'; export NV_VARS;
### opam exec -- opam env --revert | grep "^NV_VARS|^OPAM_SWITCH_PREFIX|${OPAM}"
OPAM_SWITCH_PREFIX=''; export OPAM_SWITCH_PREFIX;
NV_VARS=''; export NV_VARS;
### opam exec -- opam env --revert | grep "NV_VARS|OPAM_SWITCH_PREFIX|${OPAM}"
unset OPAM_SWITCH_PREFIX;
unset NV_VARS;
### opam exec -- env | grep '^NV_VARS|^OPAM_SWITCH_PREFIX|${OPAM}'
NV_VARS=hej!!
OPAM=${OPAM}
@@ -85,14 +85,14 @@ set -gx NV_VARS 'hej!!';
setenv OPAM_SWITCH_PREFIX '${BASEDIR}/OPAM/conffile';
setenv NV_VARS 'hej!!';
### opam exec -- opam env --revert --shell=bash | grep -v PATH
OPAM_SWITCH_PREFIX=''; export OPAM_SWITCH_PREFIX;
NV_VARS=''; export NV_VARS;
unset OPAM_SWITCH_PREFIX;
unset NV_VARS;
### opam exec -- opam env --revert --shell=zsh | grep -v PATH
OPAM_SWITCH_PREFIX=''; export OPAM_SWITCH_PREFIX;
NV_VARS=''; export NV_VARS;
unset OPAM_SWITCH_PREFIX;
unset NV_VARS;
### opam exec -- opam env --revert --shell=fish | grep -v PATH
set -gx OPAM_SWITCH_PREFIX '';
set -gx NV_VARS '';
set -ge OPAM_SWITCH_PREFIX;
set -ge NV_VARS;
### opam exec -- opam env --revert --shell=csh | grep -v PATH
setenv OPAM_SWITCH_PREFIX '';
setenv NV_VARS '';
unsetenv OPAM_SWITCH_PREFIX;
unsetenv NV_VARS;