Skip to content

Conversation

@ewoutp
Copy link
Contributor

@ewoutp ewoutp commented Nov 13, 2017

fixes #93

@ewoutp
Copy link
Contributor Author

ewoutp commented Nov 13, 2017

@matthewvon please test & review

@ewoutp ewoutp self-assigned this Nov 13, 2017
@matthewvon
Copy link

I have confirmed that code on master does NOT find /usr/local/sbin/arangod and that this branch DOES find /usr/local/sbin/arangod.

Side note: the help text on command failure only lists /usr/sbin/arangod as a potential home:

2017/11/13 10:23:43 Starting arangodb version dev, build dev
2017/11/13 10:23:43 Cannot find arangod (expected at /usr/sbin/arangod).

Might want to add " or /usr/local/sbin/arangod".

Copy link

@matthewvon matthewvon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

functionality verified. Suggest updating command line message to include new potential location.

@ewoutp ewoutp merged commit 301d3a1 into master Nov 14, 2017
@ewoutp ewoutp deleted the usr_local_sbin branch November 14, 2017 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check other places for arangod

3 participants