<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/D7671" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Albert: Do you remember why <a href="https://phabricator.kde.org/R223:f93ccd7923491c6b1412ba5cb1fe0711e44496d8" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">f93ccd7</a> was necessary? There is no bug linked and no autotest. Using just the commit message as a testcase, with <a href="https://phabricator.kde.org/D7671" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7671</a> applied and <tt style="background: #ebebeb; font-size: 13px;">watcher->addDir(fi.absolutePath())</tt> removed I can still trigger a reload with <tt style="background: #ebebeb; font-size: 13px;">rm file.txt; touch file.txt</tt>. At first sight, watching a directory when we are interested in the file feels like a workaround for bugs or missing features in KDirWatch at the time (and not needed anymore today), perhaps something like inode vs. path watchers.</p>

<p>We could think about whether <a href="https://phabricator.kde.org/R223:f93ccd7923491c6b1412ba5cb1fe0711e44496d8" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">f93ccd7</a> can be reverted in total (not sure, did not grasp the other code parts yet) in favour of <a href="https://phabricator.kde.org/D7671" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7671</a>. The former seems complicated and does not support Kate, the latter is easy to understand and supports both the original case and Kate.</p>

<p>If there is interest, I can provide logs from the ancient system gathered with <tt style="background: #ebebeb; font-size: 13px;">inotifywait -m .</tt> which show what's going on.</p>

<p>(Julian: Sorry to intrude, feel free to pick up from here.)</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/D7671" rel="noreferrer">https://phabricator.kde.org/D7671</a></div></div><br /><div><strong>To: </strong>progwolff, aacid<br /><strong>Cc: </strong>sander, rkflx, Okular, aacid<br /></div>