-
Notifications
You must be signed in to change notification settings - Fork 85
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
test/test
looks for bedrock
in the PATH
#1618
Comments
Thanks for reporting this! This seems like a reasonable change to me - would you be willing to put up a PR for it? :) |
Two bytes? C'mon… |
I know it seems silly, but I can't approve my own PR :) |
No, but you can commit it and hog all the credit for the innovation, and it will be perfectly fine with me. :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After building the server program for the first time,
bedrock
is in the top-level source directory but not yet installed. If you then saytest/test
, you get this error:This is because it is defaulting to "
bedrock
", which doesn't exist in thePATH
, as consulted byexecvp()
intest/lib/BedrockTester.cpp
.Since testing the binary before installing it is a good plan, I suggest applying this trivial patch:
The text was updated successfully, but these errors were encountered: