<div dir="ltr"><div>Hey Markus,<br><br>we did not add the DUNE_LIBS to the ALL_PKG_LIBS, but wrote the macro add_dune_all_flags in a way, that it appends both DUNE_LIBS and ALL_PKG_LIBS (in that order). This way the scenario you mention does not come up. We were not sure whether people would expect the DUNE_LIBS to be part of ALL_PKG_LIBS. If so, the find scripts should prepend the flags instead of appending them.<br><br></div><div>Best,<br>Dominic<br><br></div><div>PS: When you proposed to get rid of all the package-specific flags and macros, did you think of a mechanism like this one or something entirely new?<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 16, 2014 at 3:14 PM, Markus Blatt <span dir="ltr"><<a href="mailto:markus@dr-blatt.de" target="_blank">markus@dr-blatt.de</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<span class=""><br>
<br>
On Tue, Dec 16, 2014 at 02:51:33PM +0100, Christoph Grüninger wrote:<br>
><br>
> commit ae1cc4f56c999c8d17cb3c9d2b885706ef56a6cb<br>
> Author: Christoph Grüninger <<a href="mailto:gruenich@dune-project.org">gruenich@dune-project.org</a>><br>
> Date:   Tue Dec 16 14:51:06 2014 +0100<br>
><br>
>     [CMake] Adjust find SuperLU script modifies ALL_PKG_LIBS property<br>
><br>
>  cmake/modules/FindSuperLU.cmake | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
><br>
><br>
><br>
> diff --git a/cmake/modules/FindSuperLU.cmake b/cmake/modules/FindSuperLU.cmake<br>
> index 28f4e45..4c96e4a 100644<br>
> --- a/cmake/modules/FindSuperLU.cmake<br>
> +++ b/cmake/modules/FindSuperLU.cmake<br>
> @@ -139,4 +139,6 @@ if(SUPERLU_FOUND)<br>
>    foreach(dir ${SUPERLU_INCLUDE_DIRS})<br>
>      set_property(GLOBAL APPEND PROPERTY ALL_PKG_FLAGS "-I${dir}")<br>
>    endforeach()<br>
> +  set_property(GLOBAL APPEND PROPERTY<br>
> +    ALL_PKG_LIBS "${SUPERLU_DUNE_LIBRARIES}")<br>
>  endif()<br>
><br>
<br>
</span>Aren't the libraries ending up in the wrong order here:<br>
<br>
package dune-a depends on package b, then we first search for b<br>
and then for dune-a which results in -lb -ldunea. That will not work<br>
with static linkage.<br>
<span class="HOEnZb"><font color="#888888"><br>
Markus<br>
--<br>
Do you need more support with DUNE or HPC in general?<br>
<br>
Dr. Markus Blatt - HPC-Simulation-Software & Services <a href="http://www.dr-blatt.de" target="_blank">http://www.dr-blatt.de</a><br>
Hans-Bunte-Str. 8-10, 69123 Heidelberg, Germany<br>
Tel.: <a href="tel:%2B49%20%280%29%20160%2097590858" value="+4916097590858">+49 (0) 160 97590858</a><br>
</font></span><br>_______________________________________________<br>
Dune-devel mailing list<br>
<a href="mailto:Dune-devel@dune-project.org">Dune-devel@dune-project.org</a><br>
<a href="http://lists.dune-project.org/mailman/listinfo/dune-devel" target="_blank">http://lists.dune-project.org/mailman/listinfo/dune-devel</a><br>
<br></blockquote></div></div>