<table><tr><td style="">rkflx 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/D7594" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Only used bugzilla's quicksearch which omits NEEDSINFO bugs, so I missed this (sorry). There doesn't seem do be any new information though. I'll add a link to this review.</p>

<p>When implementing the patch, I had to decide between the version in the texlive svn and several branches of the github repo. Both are roughly equivalent to the github state if you look at bugfixes and ignore new feature development after TeXLive 2017 (in terms of code, as the actual commit history is totally different). I like that development is opening up, but there is also confusion about branches (see issue #3) and synchronisation with the svn. In the end, distros are shipping from svn, so I used that.</p>

<p>As mentioned in the summary, we could think about our options for the longterm future. For 17.12 I would suggest to go with the pragmatic approach as outlined in this review request, except you are able to invest much time in a better solution right now. The work of going through the old patches and local commits without patches was needed anyway.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7594" rel="noreferrer">https://phabricator.kde.org/D7594</a></div></div><br /><div><strong>To: </strong>rkflx, Okular, sander<br /><strong>Cc: </strong>aacid<br /></div>