barefoot
barefoot copied to clipboard
Simplify appending to env var JAVACMD_OPTIONS
In default Bash, it is not an error to refer to an undeclared (or empty) variable. In that case, it will just expand to an empty string.
So there's no need to do the prior check, you can just append the new value right away.
Also, in Bash you can safely set and export on the same command.
Thanks. I love the conciseness of this! PR #90 adds -o nounset
to this file, so when that is merged, this won't work as is. But it could still be a one-liner.
@jongiddy thanks for the heads-up!
In that case, the ${foo:-}
parameter expansion should do the trick to avoid raising error when unset.
Since it will also work even when -o nounset
is not active, I'll update this PR with the fix.