<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 #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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></blockquote>

<p>Correction: I was looking for wrong file name. config-cantorlib.h is included at several places. Looks like we simply miss #include <config-cantorlib.h> in wokrsheet.cpp. <a href="https://phabricator.kde.org/p/sirgienko/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@sirgienko</a> can you add this include and to try out your test scenarios for octave? It should work also without that add_definiton() line in CMakeLists.</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>