D7742: KDevelop projects : make use of KDirWatch optional

René J.V. Bertin noreply at phabricator.kde.org
Sat Sep 9 08:44:17 UTC 2017


rjvbb added a comment.


  >   Can't come up with proper solution right away, but I also don't like adding more checkboxes.
  
  I'd strongly advise against anything that involves automatic detection when watchers should be added or not. One thing that we might be able to do is not setting watchers on directories matching the project's exclude filter; that would address your issue with the build dir. `AbstractFileManagerPlugin::import()` would become much bigger though (finding all matching items and invoking KDirWatch::remove{Dir,File}) for them) and you'd probably also want to do this when the project filter changes.
  
  Of course the default can be OFF in set-ups where KDirWatch is known not to work properly (or be especially expensive).
  
  In defense of "my" checkbox: the "Projects" settings pane is mostly empty for now ;)

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D7742

To: rjvbb, #kdevelop
Cc: arrowdodger, brauch, zhigalin, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170909/7dc4a6b7/attachment.html>


More information about the KDevelop-devel mailing list