This doesn't compile for me:

[ 11%] Building CXX object src/engine/CMakeFiles/KF5BalooEngine.dir/indexerstate.cpp.o
In file included from /home/nate/kde/src/baloo/src/engine/indexerstate.cpp:1:
/home/nate/kde/src/baloo/src/engine/indexerstate.h:32:19: error: expected constructor, destructor, or type conversion before ‘(’ token
   32 | Q_NAMESPACE_EXPORT(BALOO_ENGINE_EXPORT)
      |                   ^

Thanks, now it compiles! I'm not sure it's working perfectly though. I updated D25447 to use this, but it's claiming that my indexer is suspended, while balooctl status disagrees.

@davidedmundson ping. This needs to make it into Frameworks 5.66 this is tagged in a week if the dependent patches are to land in Plasma 5.18.

Thanks, now it compiles! I'm not sure it's working perfectly though. I updated D25447 to use this, but it's claiming that my indexer is suspended, while balooctl status disagrees.

There were several other problems with the QML plugin which I've hopefully fixed up separately.

Now this patch no longer applies cleanly after those changes; needs a rebase.

Thanks!

Baloo::Suspended doesn't seem to be exported properly or set. Also it seems like toggleSuspendState() only ever suspends and never resumes. Looking at the code in monitor.cpp, it seems like these may be related.

@broulik discovered that this needs qRegisterMetaType<Baloo::IndexerState>(); added

Ping.

Ping.

Will do on Sunday (after tagging)

In D25743#619435, @davidedmundson wrote:

Ping.

Will do on Sunday (after tagging)

Which Sunday? ;-)

Can we please continue this over on Gitlab?