<table><tr><td style="">aacid 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/D20005">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/D20005#437053" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20005#437053</a>, <a href="https://phabricator.kde.org/p/cgiboudeaux/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@cgiboudeaux</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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/D20005#437052" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20005#437052</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);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Well, KF5I18nConfig.cmake includes KF5I18nMacros.cmake that requires Qt5Widgets.</p></blockquote>

<p>Does it? it doesn't here, or you mean "requires Qt5Widgets" because it tries to use Qt5::uic ?</p></div>
</blockquote>

<p>Qt5::uic is defined in Qt5WidgetsConfigExtras.cmake</p></div>
</blockquote>

<p>Yes, but Qt5::uic  is only used if you call KI18N_WRAP_UI, if you call KI18N_WRAP_UI is because you have .ui files in your project, if you have .ui files in your project, your project should be depending on Qt5Widgets, if it's not, it's a bug because you are directly using Qt5Widgets, so just add it there.</p>

<p>Making ki18n depend on Qt5Widgets unconditionally because a function that you may call if you use Qt5Widgets needs Qt5Widgets doesn't seem the best of the ideas to me, there's valid use cases for ki18n to not use Qt5Widgets and in those cases it'll never go thorough KI18N_WRAP_UI, so it will just work.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R249 KI18n</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20005">https://phabricator.kde.org/D20005</a></div></div><br /><div><strong>To: </strong>aspotashev, Frameworks<br /><strong>Cc: </strong>cgiboudeaux, aacid, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>