<table><tr><td style="">cgiboudeaux added inline comments.
</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/D13328">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13328#inline-69428">View Inline</a><span style="color: #4b4d51; font-weight: bold;">adridg</span> wrote in <span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:65-66</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Hi. If you're posting a review, please provide constructive input. If there are policies or guidelines about places, please link to them. If there aren't, but you would prefer a change, please describe the (outline of the) change. If you mean "there should be an additional space in the else() statement to match the text formatting from line 57", then write that.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">your else() follows if (IS_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}/po"), I thought the comment was clear.</p>

<p style="padding: 0; margin: 8px;">Your current patch means:  if there's no "po" subdir, throw a fatal error with an unrelated message.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R272 KDNSSD</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13328">https://phabricator.kde.org/D13328</a></div></div><br /><div><strong>To: </strong>adridg<br /><strong>Cc: </strong>cgiboudeaux, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>