<table><tr><td style="">winterz 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/D14563">View Revision</a></tr></table><br /><div><div><p>no objections. looks ok to me.</p></div></div><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/D14563#inline-76953">View Inline</a><span style="color: #4b4d51; font-weight: bold;">vkrause</span> wrote in <span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:1</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Note that this isn't needed for normal building, the generated files are checked in. You'd only need this when updating the data table from newer CLDR data. Rebuilding the data tables every time seems overkill, especially since the CLDR dataset is ~180MB of XML files, and new countries aren't added that often.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">ack.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R174 PIM: KContacts</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14563">https://phabricator.kde.org/D14563</a></div></div><br /><div><strong>To: </strong>vkrause<br /><strong>Cc: </strong>svuorela, winterz, kde-pim, dvasin, rodsevich, vkrause, mlaurent, knauss, dvratil<br /></div>