Markus Blatt authored 11 years ago and Christoph Grüninger committed 11 years ago

With this patch VariableSizeCommunicator now really supports a varying number of data entries
per index during communication.
(cherry picked from commit 8ce06634)

Signed-off-by: Christoph Grüninger <gruenich@dune-project.org>

Markus Blatt authored 11 years ago and Christoph Grüninger committed 11 years ago

[parallel,release] Adds a new buffered communicator that does not need to know the data amount per index when receiving.

During communication of data with a parallel DUNE grid the amount of data items
received at an index is not known at the receiving side. With this communicator
it is possible to build such a communication based on index lists at the sending
and receiving side. With this patch only fixed size of data items per index is
supported.
(cherry picked from commit dd1e8595)

Signed-off-by: Christoph Grüninger <gruenich@dune-project.org>

Markus Blatt authored 11 years ago and Christoph Grüninger committed 11 years ago

(cherry picked from commit 6c77b159)

Signed-off-by: Christoph Grüninger <gruenich@dune-project.org>

This fixes make dist.
(cherry picked from commit 3bf1f921)

Signed-off-by: Christoph Grüninger <gruenich@dune-project.org>

Steffen Müthing authored 11 years ago and Christoph Grüninger committed 11 years ago

As decided in the doodle for FS#1395, really old MPI versions will be
deprecated in the 2.3 release.

This patch adds an additional check to the MPI tests for autotools and
cmake that makes sure the user has an MPI which is at least compliant
with MPI-2.1. If the user's version is older, a warning is generated.

The test uses version macros instead of the more standard runtime
MPI version query interface to avoid problems when cross-compiling.
(cherry picked from commit ac6885f8)

Signed-off-by: Christoph Grüninger <gruenich@dune-project.org>

With this patch the USE_CMAKE environment variable is queried whether or not CMake
should be used. This can be overwritten in an options file or on the command line
using either the --cmake or --no-cmakr switch.
(cherry picked from commit cba27f049f5b5bc8b92ea86764ed1b1ff06fd9b0)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Christoph Grüninger authored 11 years ago and Markus Blatt committed 11 years ago

This fixes a Clang warning.
(cherry picked from commit bb245b8b)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Christoph Gersbacher authored 11 years ago and Markus Blatt committed 11 years ago

Make more precise statement to the user if no template specialization
of DenseMatrixAssigner is found. Also use dune_static_assert instead of
static_assert.
(cherry picked from commit d58db46d)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Christoph Gersbacher authored 11 years ago and Markus Blatt committed 11 years ago

This implementation of an IdentityMatrix does not hold any data. It
implements a reduced version of the DenseMatrix interface including a
standard C++ cast operator to a Dune::FieldMatrix. This code illustrates
the new forward capabilities of DenseMatrixAssigner.
(cherry picked from commit 64df6274)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Christoph Gersbacher authored 11 years ago and Markus Blatt committed 11 years ago

The default implementation of DenseMatrixAssigner replaces redundant
code in dense matrix constructors taking a primitive data type.
(cherry picked from commit 70bb78cd)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Christoph Gersbacher authored 11 years ago and Markus Blatt committed 11 years ago

[DenseMatrix] Introduce DenseMatrixAssigner for initializing dense matrices from arbitrary right hand sides

In order to overcome the deficiencies of istl_assign_to_fmatrix() we
propose to always use a struct DenseMatrixAssigner for initializing a
dense matrix. This class can be specialized by users for any right hand
side to be used in an assignment.  The new mechanism can bei default
implemented using existing implementations of istl_assign_to_fmatrix()
as well.
(cherry picked from commit 0725aff8)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

There was one line of diff missing in that patch. Sorry!
(cherry picked from commit 79e8fa88)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

(cherry picked from commit ff43b1ab)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Previously, copying a RemoteIndexListModifier resulted in a double
free of a pointer to a list of global indices. There was no need to
use a pointer to this, though. After this patch we store the list directly
by value. Thus the default copy constructor works but makes a rather cheap copy
of a (most often empty) singly linked list and we get rid off the non-default
destructor.
(cherry picked from commit d561f032)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

(cherry picked from commit bb51b059)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

(cherry picked from commit b9073e97)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Tobias Malkmus authored 11 years ago and Markus Blatt committed 11 years ago

Configration of dune fails if the manpages are not distributed.
(cherry picked from commit 329d2a35)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Oliver Sander authored 11 years ago and Markus Blatt committed 11 years ago

Professional software has man pages.
(cherry picked from commit 25d3fc28)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Christoph Grüninger authored 11 years ago and Markus Blatt committed 11 years ago

Properly quote variable GMP_FOUND.
Do not reuse variable GMP_FOUND.
(cherry picked from commit 317ae349)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

(cherry picked from commit 7d3a491d)

Treating directories with spaces in autotools is a can of worms.
We now check whether this situation is given and issue an error when
building with autotools.
(cherry picked from commit 7a0dfc79)

With this patch dunecontrol will work even if the path to the module
contains spaces. Unfortunately, the tests for dune modules will still
not work as I could not figure out how to use quoting in the CPP_FLAGS.
(cherry picked from commit eaedbeea)

(cherry picked from commit 5cc7b5b8)

Christoph Grüninger authored 11 years ago and Markus Blatt committed 11 years ago

At least Clang 3.2 complains.
(cherry picked from commit d4014751)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

This is a follow up to the previous patch. It corrects the situation
for new modules created with duneproject.
(cherry picked from commit a0f4d2c6)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Torbjörn Klatt authored 11 years ago and Markus Blatt committed 11 years ago

Previously the module path was a relative to the install
directory. Because of this dependant modules did not find
the macros of dune-common and cmake configure did not succeed.
With this the CMake package configuration file contains the
full path and the macros are found (even if there is no local
dune-common lying around).
(cherry picked from commit db6f61c2)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

Somehow MPI was never activated for the headercheck. This patch fixes this.
(cherry picked from commit fbfa416b)

Signed-off-by: Markus Blatt <markus@dr-blatt.de>

The config.h entry for ENABLE_UMFPACK should be in the same
module as the test for UMFPack, to avoid future conflicts with
multiple modules running the test for umfpack.

DenseVector is the base class of a CRTP. This is kind of an abstract
class that works only if it can be casted into a derived type. Copying
the base class is pointless.

This patch marks the copy constructor of DenseVector private (not even
implementing it), so that it cannot be called anymore. Once we have a
DUNE_DELETE macro, we should use it, here.

At the same time, the default constructor is marked protected, so that only
derived classes can instantiate a DenseVector. Notice that this does not
prevent the user from building a derivced class which does not pass
itself as implementation to DenseVector. It is (to my knowledge) not possible
to make CRTPs completely fail-safe.

Cmake by default added a dependeny variable for each headercheck
target. Given the amount of such targets and the fact, that these
dependencies should not be changed by the user, this is unwanted
behaviour. The patch fixes this by removing the variable from the
cache.

The removed lines would be necessary if the included headers
or libraries would be tested. As this is not the case, they can
be safely removed.
The way they were used, every test tried to link to the libraries
amd and umfpack causing every test to fail if the libraries are
not present. The changed declaration of CMAKE_REQUIRED_* should
be encapsulated together with the test inside
  include(CMakePushCheckState)
  cmake_push_check_state()
and
  cmake_pop_check_state()
to ensure that the definitions do not leak the test.

Dominic Kempf authored 11 years ago and Markus Blatt committed 11 years ago

Using variables as part of regex does not work with special characters.
Therefore previously the headercheck failed during configure if e.g.
the modules were located in $HOME/C++/. This patch fixes this by not
using regex to strip the location of the modules from the path any more.

This log message was amended by the pusher.