<div dir="ltr"><div><div><div>Hey Christoph, hey Jö,<br><br></div>I know it fails on non-make build systems - thats why the CMAKE_GENERATOR variable is matched with ".*Unix Makefiles.*" before applying the changes.<br><br></div>The branch name is accidental. I will not change it for now, to keep the link in my email alive. But if it is not merged in the future, I will rename it correctly.<br><br></div><div>Thanks for pointing out the '/' Jö. I fixed it, but I am still puzzled why it worked.<br></div><div><br></div>Best,<br>Dominic<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 26, 2014 at 7:49 PM, Jö Fahlke <span dir="ltr"><<a href="mailto:jorrit@jorrit.de" target="_blank">jorrit@jorrit.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Dominic!<br>
<br>
Looking at the latest branch<br>
(<a href="http://cgit.dune-project.org/repositories/dune-common/commit/?h=p/dominic/allow-extra-cppflags-in-cmake&id=f578610b3648e55bf7e5f7aaaef6c3c46468533a" target="_blank">http://cgit.dune-project.org/repositories/dune-common/commit/?h=p/dominic/allow-extra-cppflags-in-cmake&id=f578610b3648e55bf7e5f7aaaef6c3c46468533a</a>),<br>
in the line<br>
<br>
+ exec_program(chmod ARGS "+x ${CMAKE_BINARY_DIR} compiler.sh")<br>
<br>
there appears to be a / missing in front of compiler.sh.<br>
<br>
Am Fri, 26. Sep 2014, 18:59:22 +0200 schrieb Christoph Grüninger:<br>
<span class="">> My only complaint is the name of the branch. cpp is the C preprocessor.<br>
> This is a common misnomer in our build-system code, but we should get<br>
> rid of it.<br>
<br>
</span>Indeed the distinction between CXXFLAGS and CPPFLAGS is dissolved, but I don't<br>
think it matters for this use case.  You have to be careful though to<br>
explicitly override any flags that are set by default.  (E.g. with automake,<br>
if CXXFLAGS=-O3 CPPFLAGS=-DNDEBUG was the default, specifying CXXFLAGS=-Wall<br>
CPPFLAGS="" on the make command line would also turn off optimizations and<br>
enable assertions; now you explicitly have to specify EXTRA_CXXFLAGS="-Wall<br>
-O0 -UNDEBUG").<br>
<br>
Regards,<br>
Jö.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Jorrit (Jö) Fahlke, Institute for Computational und Applied Mathematics,<br>
University of Münster, Orleans-Ring 10, D-48149 Münster<br>
Tel: <a href="tel:%2B49%20251%2083%2035146" value="+492518335146">+49 251 83 35146</a> Fax: <a href="tel:%2B49%20251%2083%2032729" value="+492518332729">+49 251 83 32729</a><br>
<br>
It is my conviction that killing under the cloak of war is nothing but<br>
an act of murder.<br>
-- Albert Einstein<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><br></div>