<div dir="ltr"><div><div><div><div>Hey Christian,<br><br></div>I don't get exactly what you are proposing here, because<br></div>1) the script doesn't has to be modified, because it is generated by the build system upon configuration.<br></div>2) Changing the compiler without reconfiguring isn't supported by cmake anyway.<br><br></div><div>Andreas, I am hesitant about merging, as I think, there might be some scenarios, where this will bite you. I am not (yet) able to name one such scenario though. But you are right, users not setting the flag are 100% free of side effects.<br></div><div><br></div>Dominic<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 24, 2014 at 2:25 PM, Christian Engwer <span dir="ltr"><<a href="mailto:christian.engwer@uni-muenster.de" target="_blank">christian.engwer@uni-muenster.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">One comment...<br>
shouldn't we do it in such a way that the compiler command is<br>
  compiler.sh $CXX<br>
?<br>
<br>
In this case it would not be necessary to modify the script when<br>
specifying the compiler, or did  I read the commit wrong?<br>
<br>
Christian<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Sep 24, 2014 at 01:20:03PM +0200, Dominic Kempf wrote:<br>
> I made a typo (it should be CMAKE_FLAGS) in the "manual".<br>
><br>
> I amended that and then realized, this changes the ref. please see this<br>
> link, instead of above:<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>
><br>
> Dominic<br>
><br>
> On Wed, Sep 24, 2014 at 11:58 AM, Dominic Kempf <<a href="mailto:dominic.r.kempf@gmail.com">dominic.r.kempf@gmail.com</a>><br>
> wrote:<br>
><br>
> > Hey Dune,<br>
> ><br>
> > the feature to add compile flags when running make seems to be of great<br>
> > importance to a lot of autotools users. This is conceptionally impossible<br>
> > to integrate in cmake. We have thought of a hack, that does the trick<br>
> > anyway.<br>
> ><br>
> > I am not sure whether this should actually be merged, as it is rather<br>
> > dirty. On the other hand, it doesn't do any harm, as it does nothing if not<br>
> > specifically switched on. For details on how to use it, check the commit:<br>
> ><br>
> ><br>
> > <a href="http://cgit.dune-project.org/repositories/dune-common/commit/?h=p/dominic/allow-extra-cppflags-in-cmake&id=dbccc31548c3347d21712d3f808ad4ccb4895b5d" target="_blank">http://cgit.dune-project.org/repositories/dune-common/commit/?h=p/dominic/allow-extra-cppflags-in-cmake&id=dbccc31548c3347d21712d3f808ad4ccb4895b5d</a><br>
> ><br>
> > Enjoy!<br>
> > Dominic<br>
> ><br>
<br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<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>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Prof. Dr. Christian Engwer<br>
Institut für Numerische und Angewandte Mathematik<br>
Fachbereich Mathematik und Informatik der Universität Münster<br>
Einsteinstrasse 62<br>
48149 Münster<br>
<br>
E-Mail  <a href="mailto:christian.engwer@uni-muenster.de">christian.engwer@uni-muenster.de</a><br>
Telefon <a href="tel:%2B49%20251%2083-35067" value="+492518335067">+49 251 83-35067</a><br>
FAX             <a href="tel:%2B49%20251%2083-32729" value="+492518332729">+49 251 83-32729</a><br>
</font></span></blockquote></div><br></div>