- moved some parts relating to the venv out of PythonCommonMacros to Virtualenv
- removed PythonMacros file
- use an external active env instead of generating a new internal one.
  Also added some checks to avoid a user mixing up different builds within the same env
- finally made some changes on how and when the dune-py module is regenerated

some minor fixes

[testing] ci with index-url set

remove extra-index-url from pip install

add a stopgap solution unti ci repo up and running again

we need pip>=21 so that 'make', 'make install_python', 'make' does not
mess up the packages in an external env (with older versions of pip
the last 'make' wouldn't correctly remove the non editable install).

pep 42: use implicit namespace packages which make it possible to combine subpackages using editable (source) and non editable (pypi)

use clang 10-20 for python test since some other parallel tests fail with the clang 10-17 toolchain

fix an issue with a double installation of the dune module in the case
of using packages (there was a 2.8.0 version and a dev version being added to the venv)

minor improvement to plotting

Add a 'flag' argument to the 'dune_python_install_package' command which contains cmake variables to store in metadata file.
This is to be used to fix for example the compiler to use for dune-py but that hasn't been added to the builder yet.
Also changed builddirs output to contain ';' separator (using a cmake string instead of a list).

added 'cmake flags' from metadata file to CMakeLists.txt.template

correctly throw exception if dune-py not available in assertHave

been constructed.
The __init__.py file in dune.grid uses 'haveAssert' to check if 'AlbertaGrid'
is available. This checks 'config.h' in dune-py which caused an error.
The fix here simply return 'False' in case dune-py/config.h is not available.

correctly use the tagfile '.noconfigure' to make sure dune-py is rebuild
each time a new dune module is installed.
To avoid a 'recursive include' issue moved the generation of dune-py based on the
'template' to the 'Builder' (where it was before).

Also fixed a bug (which was partially a feature avoiding recursive includes up until now):
there was a script dune/generator/builder.py which contained the 'Builder' class and
in dune/generator/__init__.py we setup a variable 'builder' which then shadowed the script.
This did then cause problems in one place. Renaming the 'builder.py' script then demonstrated
the issue with the recursive import.

use *.template for dune-py template files which should be copied without
the ending, e.g., CMakeLists.txt (now CMakeLists.txt.template)
Also renamed dune.module to dune.module.template to be certain that it will
not be picked up by mistake

add cmake install command for the cmake_metadata files and the files in the
dune-py template to make sure that skbuild adds them correctly to the wheel

check for matplotlib in dune.plotting

Partial fix for export of build dirs for all deps modules to metadat file.
In previous version only first builddir was written.
Issue with this version: space is used to separate the directories.
Also make dune-py depend on modules without their own python package if another module
(with bindings) depends on it.

minor fixes

Dominic Kempf authored 4 years ago and Andreas Dedner committed 3 years ago

Install CMake scipt

Dominic Kempf authored 4 years ago and Andreas Dedner committed 3 years ago

Slightly refactor data processing and fix the common-install use case

Dominic Kempf authored 4 years ago and Andreas Dedner committed 3 years ago

All testing is done in the virtualenv.

Dominic Kempf authored 4 years ago and Andreas Dedner committed 3 years ago

It is not really necessary in the new interface.

Dominic Kempf authored 4 years ago and Andreas Dedner committed 3 years ago

This reduces the number of redundant reruns of the env installation.

Revert "Merge branch 'feature/modern-cmake-documentation-generator' into 'master'"

See merge request core/dune-common!1024

This reverts merge request !1020

Remove deprecated file DuneMPI.cmake

See merge request core/dune-common!1023

Add Changelog entry

Remove deprecated dune_list_filter and deprecate file DuneCMakeCompat.cmake

See merge request core/dune-common!1022

Extract dune_add_library from DuneMacros

See merge request core/dune-common!1019

Add generator for modern buildsystem documentation

See merge request core/dune-common!1020

Improve Find<Module> CMake package documentation

See merge request !1021

[python][typeregistry] _typeName/_includes -> cppTypeName/cppIncludes

See merge request !980

Timo Koch authored 3 years ago and Andreas Dedner committed 3 years ago

Underscore prefixed variables in Python are often by convention internal variables
that should not be accessed from outside the class. These attributes are frequently
used in Python-side C++ code generation code and have to be accessed to write code generators.
Linters like pylint complain about the access of these protected attributes.
This changes them to be part of the public interface (by naming convention).

Include deprecation period

Add flag NO_MODULE_LIBRARY to dune_add_library

See merge request !1014

Remove the cmake variable DUNE_DEFAULT_LIBS

See merge request !1009

Remove the cmake module include CMakeParseArguments

See merge request !1008