Updating code to use newer CMake MPI variables.

FindMPI.cmake has deprecated MPI_LIBRARIES, MPI_INCLUDE_PATH variables (since
before 2.8.5) and instead exposes MPI_C_LIBRARIES, MPI_C_INCLUDE_PATH variables.
The old/deprecated variables work very unreliably especially with mpi
implementation that require multiple libraries e.g. openmpi. Hence updating all
code to use the newer MPI variables.

Change-Id: I68e4a72a144e38964111671b0ea6da2314473dbd
Fixing windows MPI build issue with Cosmo.

When building with OpenMPI, it's required that any library linking with mpi
libraries also link with ws2_32 and Psapi libraries for socket API used by
OpenMPI. vtksys cleanly adds rules to depend on these libraries. Hence we add a
depdency on vtksys to ensure appropriate dependenices are brought in.

Change-Id: I10a84d51fc8c1fecc587113ba5d0269ac451c892
ENH: 100% coverage for XMLFileOutputWindow.

A new unit test provides 100% coverage.

computeCodeCoverageLocallyForOneTest.sh Common/Core -R TestXMLFile

Change-Id: I9565397d7d06cdfbc69c5f8afd6bfe650dc6c7eb
BUG 13443: Fix crash when vtkMath.h is wrapped on FreeBSD.

The isinf() and isnan() macros on FreeBSD are defined in terms of
the isinf() and isnan() library functions.  A macro expansion bug
in the VTK 5.10 wrappers caused these macros to expand recursively,
resulting in a segfault in the wrapper tools.  The macro expansion
bug is fixed in VTK 6, but VTK 5.10 requires the vtkMath IsNan()
and IsInf() inline definitions to be blocked.

Change-Id: Ifc4ce554fd7273367f3fa6c6dad3b26dfa7871d4
BUG 13198: Newer Tcl/Tk packages need static_cast with ckalloc.

The ckalloc() function that Tcl uses for memory allocation has
been changed in Tcl 8.6 to return "void *" instead of "char *".

Change-Id: I7a09bd1e247748c4f4b4b017ca26f1cc8130463d
Merge topic 'remove_deprecated_mpi_variables' into master

5528c7c6 Fixing windows MPI build issue with Cosmo.
4e7bdc3d Updating code to use newer CMake MPI variables.
Merge topic 'Coverage_XMLFileOutputWindow' into master

65741630 ENH: 100% coverage for XMLFileOutputWindow.
Add `vtkDataObjectTree::NewTreeIterator()` method.

This is a convenience so that you don't have to
downcast the composite dataset iterator returned
by `NewIterator()` to call methods like `VisitOnlyLeavesOff()`
or `TraverseSubTreeOff()`.

Change-Id: Ibe53476e1d040ccdb29b27bbdf5090bfbe40168c
Make Bill less unhappy; test more.

Change-Id: Ib74174d7767a68740acee8a403cc0d7402c6a4dc
Fixing bug in if logic.

Fix bug introduced by commit e5102f45. The if() condition wasn't changed
correctly by that commit resulting vtkPExodusIIReader always producing empty
data.

Change-Id: Ifc5894e2978be68bd82fef0ceedfeff9cba4bb1f
Fix parallel piston test on linux

It was throwing X errors when the interactor started in -I mode.
Now it doesn't.

Change-Id: I878ac5d56c751f3f6660ebe1e49d0836304502b1
Merge topic 'composite-tree-iterator-sugar' into master

2ea4b12b Make Bill less unhappy; test more.
6478a7b3 Add `vtkDataObjectTree::NewTreeIterator()` method.
BUG: Fix GLXBadDrawable when using multiple QVTKRenderWindowInteractor

The vtkRenderWindow was never finalized which caused GLXBadDrawable
errors on application exit if using multiple QVTKRenderWindowInteractor.
This patch provides a cleanup mechanism to finalize the vtkRenderWindow
before its destruction/close preventing GLXBadDrawable errors. This
patch works in both embedded and non-embedded cases.

This patch also provides a test case for QVTKRenderWindowInteractor.

DDM: commit manufactured to force dashboard tests

Change-Id: Ie86d17fe7648868f8f1d5e4b00f52419d5f35a5b
Make the QVTKRenderWindowInteractor more widely compatible.

Fix compatibility issues with recent commit 5316dff1.  It used the new
"import as" syntax that isn't supported on python 2.5 and earlier.
It also used a new signal/slot mechanism that is only available in
PyQt 4.5 and later, even though the old signal/slot mechanism was
still being used elsewhere in the file (mixing old and new ways
of doing the same thing in a single file makes the code hard to read).

The test TestQVTKRenderWindowInteractor.py did not draw anything,
because it didn't connect the renderer to the render window, nor
did it initialize the interactor.  It also used two-space indentation
in some places and four-space indentation in others.

Change-Id: Ie41798554cce2d1ffe5705ec198d191d9e18cb08
Merge topic 'amr_streaming' into master

ef912324 Fixing bug in if logic.
Merge topic 'fix_piston_tests' into master

f72246fa Fix parallel piston test on linux
Merge branch '13443-freebsd-fixes' into release

Change-Id: I2448775ea1e97fc903402d498c524b41d9f4f260
Merge branch 'release'

Change-Id: Ie69c9b75796774cd7b607cd5e7f097a5f912b2a6
Link against MPI_CXX_LIBRARIES when present.

When buildling with OpenMPI (and others) Cxx code needs to also link against the
MPI_CXX_LIBRARIES. This fixes that issue.

Change-Id: Ie4ac255c77e254cf9b788c5bbc444e0baec0368c
Merge topic 'remove_deprecated_mpi_variables' into master

0f40060c Link against MPI_CXX_LIBRARIES when present.