-
Notifications
You must be signed in to change notification settings - Fork 442
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
Fixing multiple build issues on non-mlnx systems #21
Conversation
shamisp
commented
Nov 10, 2014
- Autotools scripts were updated
- Fixing multiple warnings in gtests
- Other minor issue
Please review and test on your systems. |
Test FAILed. Build Log
Test FAILed. |
Test FAILed. Build Log
Test FAILed. |
I don't think these tests are already available on master: It is part of #20 |
Test FAILed. Build Log
Test FAILed. |
Test FAILed. Build Log
Test FAILed. |
@yosefe - I think I addressed all you comment. In the last commit I add "HAVE_CONFIG_H" everywhere BUT since it is must to have, I think we have to remove the protection everywhere. The config.h file is not optional. |
Test FAILed. Build Log
Test FAILed. |
Test PASSed. |
@@ -1,7 +1,7 @@ | |||
#!/bin/sh | |||
# | |||
# Copyright (C) Mellanox Technologies Ltd. 2001-2011. ALL RIGHTS RESERVED. | |||
# | |||
# Copyright (c) 2014 Copyright (c) UT-Battelle, LLC. 2014. ALL RIGHTS RESERVED. |
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.
This change is noise. I will revert copy of this file to "master" version
Test PASSed. |
Test PASSed. |
👍 |
* Autotools scripts were updated * Fixing multiple warnings in gtests * Other minor issue * Making the code config.h compliant
Test PASSed. |
On my system I see the following Failure:
Is it ok ? |
+1 from @yosefe |
Fixing multiple build issues on non-mlnx systems
TEST/APPS/IO_DEMO: Merge iodemo changes from openucx:master branch.
…n-via-ep Separate receive operations based on the Endpoint it used with