Just like for the autotools.

For HAVE_METIS we do not use the ENABLE trick. It is therefore either defined or
not. Therfore using "#if HAVE_METIS" is a bug. This patch fixes this.

There were problems with repartition.hh when using the original parmetis. The cause was
that for scotch we need to require both parmetis.h and metis.h, which broke backwards
compatibility for the original parmetis (which worked without metis, too). With the patch
we check whether parmetis.h is from scotch and if that is the case also require metis. If not,
metis suffices.

Reformat whole CMakeLists.txt file for better readability.

GCC warned about ambiguity between builtin operator*=(double&, int)
and FieldVector<double,i>::operator*=(int), but at least picked one
and just emitted a warning. Clang instead errored out.

To fix this this patch explicitly uses a floating point dampening
factor in the additive version of the AMG.

Fixes flyspray #1456 
https://dune-project.org/flyspray/index.php?do=details&task_id=1456.

To test the implementation improved the test to not only include diagonal block matrices on the
diagonal, because with diagonal left/right multiplication does not make a differece.

The warning has to be silenced by using DUNE_UNUSED_PARAM in dune-common.
This fixes FS#1431.

It was moved to dune-common in Dune 2.3.

Arne Morten Kvarving authored 11 years ago and Markus Blatt committed 11 years ago

Since revision 8df1d996 overlappingschwarz
with UmfPack was broken due to renaming an internal variable accessed by a friend.
This patch fixes this by adding a new private method to access the internal matrix
of solver and uses that for the assembly.

This the customized rest of Rene's patch. To make doxygen happy
we use DUNE_UNUSED_PARAMETER.

Rene Milk authored 11 years ago and Markus Blatt committed 11 years ago

This is part of the patch provided by Rene in Flyspray 1355.

Fixes flyspray https://dune-project.org/flyspray/index.php?do=details&task_id=1393
(not stopping if there is no convergence).
In addition each residual is only printed once now.

Some cmake specific dirs have been deleted in commit c66db537
but not removed from configure.ac resulting the build process to fail for the autotools.

Previously, the macros were installed to $(datadir)/cmake/. As we have
some macros in DUNE that might also be in other packages
this may cause conflicts in package managers. With this patch we install
them to $(datadir)/dune/cmake to resolve this issue. Together with the
last patch this will resolve problems  outlined in flyspray
task 813
https://dune-project.org/flyspray/index.php?do=details&task_id=813
I apologize for givin the wrong task number in the previous patch.

Previously, the macros were installed to $(datadir)/aclocal. As we have
some macros in DUNE that are also installed for some Linux distributions
this caused conflicts in package managers. With this patch we install
them to $(datadir)/dune/aclocal to resolve this issue outlined in flyspray
task 1409
https://dune-project.org/flyspray/index.php?do=details&task_id=1409

Previously, dependencies were not correctly searched for as
required packages. This patch fixes this.

For generating the config.h file we use the HAVE_* variables instead
of the CMake ones *_FOUND, Unfortunately, we sometines to not set
HAVE_* to the value of the *_FOUND vriable but to its name. Usually
we only query whether these variable are set at all. While
this often (except for psurface) does not pose a problem, it is semantically
wrong. Therfore this patch fixes this.

Kudos to Ansgar for pointing to this problem.

Arne Morten Kvarving authored 11 years ago and Markus Blatt committed 11 years ago

After adding UMFPack support, the printed log message was "Using DIRECTSOLVER"
which irritated users. With this path the actual direct solver will be
printed.