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
Fix poco 1.6.0 warnings #144
Conversation
Oddly enough the last build failed on a couple of platforms because of the git-checkout issue again. The same checkout command worked well on the other platforms. |
Well the changes are obviously fine but just to make sure I'm going to try with a 'retest this please'. Not sure if it will work when I write it... |
retest this please |
1 similar comment
retest this please |
Thanks Harry for re-starting the build. For some reason the bug in the Jenkins plugin really likes this pull request. It failed again on 3 platforms! |
retest this please |
All looks good and the builds went through at last. |
…p0_warnings Fix poco 1.6.0 warnings
Both Poco/Platform_POSIX.h and Code/Mantid/Framework/Kernel/inc/MantidKernel/WarningSuppressions.h define the macros GCC_DIAG_ON(x) and GCC_DIAG_OFF(x). The compiler warnings can be suppressed by undefining both macros before they are defined in WarningSuppressions.h.
testing: review changes and check tests. Testing on a machine with poco 1.6.0 installed is not necessary.
http://trac.mantidproject.org/mantid/ticket/10977