Convert some libraries from static to dynamic

RShaw026 commented 1 year ago

Description and Context

When used as an external library in BACI, some linker issues occurred because of the use of a static library. Therefore, mirco_library target is converted to dynamic library and OpenMP dependency in added to it. The ideal case would be to make mirco as a dynamic library, but I was facing some issues regarding that. So I will open an issue regarding it. Merging this PR is required to move ahead with our BACI/MIRCO integration. Also, -Werror compiler flag is added to treat warnings as errors in the pipeline.

Related Issues and Pull Requests

Closes
Blocks
Is blocked by
Follows
Precedes
Related to
Part of
Composed of

How Has This Been Tested?

ctest is working. mirco executable is working. mirco_library is used in BACI in my local, and it is working.

Checklist

[ ] My commit messages mention the appropriate GitHub issue numbers and are longer than one line where appropriate.
[ ] I have added/updated documentation where necessary.

Additional Information

Interested Parties / Possible Reviewers

@mayrmt

maxfirmbach commented 1 year ago

I had a look and I'm fine with the changes, but first I would really like to have a second opinion of someone who has a better understanding of CMake and how things work together.

RShaw026 commented 1 year ago

To avoid printing the Trilinos configuration messages while building BACI, I have moved them inside the TRILINOS_IN_MIRCO flag. These things will help us to integrate MIRCO into BACI; however, the CMakeLists in MIRCO need to be redesigned.

mayrmt commented 1 year ago

Let's with this PR until PR #78 has been merged.

imcs-compsim / MIRCO