trailing whitespace
remove compiler warnings warn_unused_result about fscanf
trailing whitespace
assert that system command gets executed alright
trailing whitespace
char name[] => const char name[] in the helper functions

This avoids compiler warnings about deprecated char conversions.
trailing whitespace
assert that system command gets executed alright
trailing whitespace
make sure that Python.h is the first included system header
fix warning about deprecated char conversion
fix some format specifiers
Ifpack2: ILUT: Removed unnecessary state, and improved public docs.

For the Milestone review, it will be helpful to be able to say that
Ifpack2 implements incomplete factorizations with RCM ordering.  This
commit is a preliminary pass over the ILUT code just to learn more
about it and to correct any obvious things that need correcting.
Ifpack2: LocalFilter: Removed excess state, and improved public docs.

I'm revisiting LocalFilter, because for the Milestone review, we should
be able to say that incomplete factorizations work as subdomain solvers.
Ifpack2 uses LocalFilter to restrict a matrix to locally owned rows and
columns, so that it is legitimate to use a linear solver (the subdomain
solver) on that matrix.
Kokkos: AltSparseOps: Removed unnecessary std::string constructions.

The TEUCHOS_TEST_FOR_EXCEPTION_CLASS_FUNC macro uses a string (that
it assumes to be in scope) to add the method's name to its exception
message.  This string does not need to be an std::string; it can be
a const char[].  Matt Bettencourt found that in some cases, the cost
of std::string construction was showing up for certain Tpetra methods.
As a result, I always convert these std::string to const char[] when-
ever I see them.

Build/Test Cases Summary
Enabled Packages: Kokkos, Tpetra, Belos, Ifpack2
0) MPI_DEBUG => Test case MPI_DEBUG was not run! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Test case SERIAL_RELEASE was not run! => Does not affect push readiness! (-1.00 min)
2) MPI_DEBUG_SS => passed: passed=143,notpassed=0 (3.27 min)
3) SERIAL_RELEASE_SS => passed: passed=125,notpassed=0 (0.31 min)
Other local commits for this build/test group: f04b2a1, 8873b37
Xpetra: Adding fusedImport/fusedExport to MatrixFactory.

Not just CrsMatrixFactory.
MueLu: More fusedImport changes.
MueLu: RebalanceAcFactory Epetra fix.
Xpetra: Fixing Xpetra::EpetraCrsMatrix initialization.

100% my bad.
Triutils: Removing compiler warning.