<table><tr><td style="">arojas 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/D15091">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/D15091#316150" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15091#316150</a>, <a href="https://phabricator.kde.org/p/rdieter/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@rdieter</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Why are you simply not using all  PYQT_CONFIGURATION["sip_flags"]?</p></div>
</blockquote>

<p>Not sure if we really want to do this. For instance, bindings would get tagged with the Qt version that PyQt was compiled with, not with the one KF5 is being compiled with (not sure if that would cause any actual issue)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15091">https://phabricator.kde.org/D15091</a></div></div><br /><div><strong>To: </strong>arojas, Frameworks, bruns<br /><strong>Cc: </strong>rdieter, kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns<br /></div>