<table><tr><td style="">asemke added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11967">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D11967#240772" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11967#240772</a>, <a href="https://phabricator.kde.org/p/aacid/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@aacid</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>hmmm, this is weird, i don't really need this, the current code works for me</p>

<p>MESSAGE("LibSpectre_FOUND: ${LibSpectre_FOUND}")</p>

<p>just after the find_package says <br />
 LibSpectre_FOUND: TRUE<br />
 here when running on a clean build dir.</p>

<p>But it is true that ${LIBSPECTRE_FOUND} also prints true.</p></div>
</blockquote>

<p>This is really strange. I don't see nowhere in cantor/cmake/FindLibSpectre.cmake (not sure this file is really used by cmake) the variable LibSpectre_FOUND. Since cmake's variables are case sensitive I'd assume LIBSPECTRE_FOUND is the only correct one. Your cmake is newer than the version that I'm using. But this worked in cantor already in the past for sure, also with older versions of cmake...</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>About the add_definitions you should not need it, the generated<br />
 src/lib/config-cantorlib.h<br />
 already has that define, just make sure you include that file wherever you need it.</p></blockquote>

<p>I don't see any includes for config-cantrolib.h in the code. Maybe this is added somewhere on the cmake-level. Some parts of the current code are aware of the definition of WITH_EPS, e.g. Expression::setResult(). Some other parts like Worksheet:: loginToSession() don't "see" this variable and require now either that include you mentioned or a preprocessor definition. Since #ifdef WITH_EPS is used in several different places in Cantor's code it will take some time now to test all those places. Given the current timeline/deadline for 18.04 I'd rather go with a preprocessor definition via add_defitions() - this is safer now and also more clean and clear, at least to me, since it's straight -forward to see where it's defined.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R55 Cantor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11967">https://phabricator.kde.org/D11967</a></div></div><br /><div><strong>To: </strong>asemke, Cantor, aacid, apol, sirgienko, filipesaraiva<br /><strong>Cc: </strong>aacid, apol, KDE Edu, narvaez<br /></div>