michaelh added a comment.

  Sorry I took so long to respond. I needed to get the tools ready to see whats 
going on (D11285 <https://phabricator.kde.org/D11285> and D11287 
<https://phabricator.kde.org/D11287>) .
  Wrt this patch
  
  - You forgot to open the database in ReadWrite mode.
  - To test your code I changed that locally. After that a lot of `Q_ASSERT`s 
got in the way. I gave up after the third.
  - I wonder how you can run this. Please make sure you build the `debug 
target`. Comment out the respective `Q_ASSERT`s and mark them with a `FIXME`.
  
  Please incorporate this patch into D11285 
<https://phabricator.kde.org/D11285> and D11287 
<https://phabricator.kde.org/D11287>. It separates the concerns of controlling 
indexing and manipulating the database. Also database manipulation is 
experimental while `balooctl` stuff is not.
  It would be great if we could test the effect of this command. I have no idea 
how this can be done. Maybe you have one?
  
  > I was going to expose this functionality on the DBus interface and 
integrate it into the check command, but the lack of external mount enumeration 
makes this a little risky at the moment.
  
  I've been playing around with mount enumeration in the sanitizer class. It's 
feasable. Wait for a patch coming up soon.
  
  Personal note:
  Even after dealing with this patch for some time I still get confused by the 
terms 'prune' and 'purge'. This may be my personal problem or a general problem 
for non-native english-speakers. I don't know. For me it would be easier if 
this command would be called just `clean`.

REPOSITORY
  R293 Baloo

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

To: smithjd, #baloo, #frameworks, michaelh
Cc: michaelh, ashaposhnikov, astippich, spoorun, nicolasfella, ngraham, 
alexeymin

Reply via email to