Closed drewkerrigan closed 8 years ago
Silly shells.
Would you mind using 'id -un' instead of whoami as I'm not sure whoami is IEEE1003.2/POSIX.2 (but don't have a copy of it handy to be sure).
@jonmeredith looks like WHOAMI is used in a few spots. Are you thinking about comprehensive change, or just for the localized to the HOME var?
POSIX.2 is reportedly very similar to the Single UNIX Specification, where it appears who am i
is specified but whoami
is not.
Haha, teach me to look at just the diff and put my foot in it. If we're using whoami in other places successfully on our platforms I'm less concerned. Ideally we'd standardize on SUS/POSIX.2 commands.
Just curious, do we write to $HOME, anywhere in the code?
@jonmeredith whoami has been used for quite some time in this script. I agree it should probably be changed to id -un
, but perhaps for another PR, like this one: https://github.com/basho/node_package/pull/186 :-)
@sargun I wasn't able to find any other places writing to the HOME env variable in the release code for Riak anyway, nor did I find anything reading from it. I think erlexec
is the primary piece that requires it, which is used in a few of the scripts generated by this repo and rebar.
create jira issue
Can this be prioritized in the next set of DevOps work?
_[posted via JIRA by Derek Somogyi]_
this is merged [where is the close ticket button?]
_[posted via JIRA by David Andersen]_
Related issue here: https://github.com/basho/cuttlefish/issues/193
"erlexec requires the HOME variable to be set.
The way that this manifests itself during normal Riak startups, if $HOME isn't set:
Can we automatically set $HOME, if it's not set already?"
I've addressed this with the following addition to
env.sh
: