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
add warning for known problematic env vars #72
Conversation
24a4779
to
8456029
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks okay. Two fixes needed. I still worry each time I see the rspect code being significantly longer than the code you add to newinstall.
scripts/newinstall.sh
Outdated
|
||
if [[ ${#problems} -gt 0 ]]; then | ||
n8l::print_error "$(cat <<-EOF | ||
WARNING: the following environment variables are defined that will effect |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"affect"
scripts/newinstall.sh
Outdated
|
||
n8l::print_error "$(cat <<-EOF | ||
|
||
It is recommend that they are undefined before running this script. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"recommended"
e9231f9
to
5210a76
Compare
I'm not sure why? |
Ugh. This needs a rethink as |
Replaced with declare.
5210a76
to
83672a7
Compare
83672a7
to
e93ffac
Compare
No description provided.