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
tools: FreeBSD basic getopt does not use --options #21148
Conversation
src/tools/setup-virtualenv.sh
Outdated
@@ -17,6 +17,11 @@ | |||
|
|||
SCRIPTNAME="$(basename $0)" | |||
PYTHON_BINARY="python2.7" | |||
if [ `uname` == FreeBSD ]; then | |||
GETOPT=/usr/local/bin/getopt |
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.
put /usr/local/bin/getopt
in double-quotes?
src/tools/setup-virtualenv.sh
Outdated
if [ `uname` == FreeBSD ]; then | ||
GETOPT=/usr/local/bin/getopt | ||
else | ||
GETOPT=getopt |
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.
put getopt
in double-quotes? I know it's not striclty necessary - just to unify the coding style.
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.
@smithfarm
Sure, no problem.
c5ef2aa
to
8f65db3
Compare
retest this please |
retest this please. |
@smithfarm @tchaikov |
Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
8f65db3
to
877d856
Compare
retest this please. |
Failed run-rbd-unit-tests.sh Failed unittest_journal (crashes on assert(!m_metadata->is_initialized()) |
retest this please |
@smithfarm this PR compiles. so i guess it's good to merge, am i right? |
Signed-off-by: Willem Jan Withagen wjw@digiware.nl