Hi, +1 to what Carl said. Right now my Elisa barely needs any time to read my music library, presumably because Baloo already has it indexed. It'll be quite a shame if apps did their own indexing, wasting time and power.
The effects (of removing Baloo support) are even more pronounced if we consider that a majority of Elisa users are running Plasma as well. (Citation needed, I know, but IMO it's not such a risky assumption.) Regards, Shantanu On Sun, Nov 5, 2023, 11:53 PM Carl Schwan <c...@carlschwan.eu> wrote: > On Sunday, November 5, 2023 6:01:38 PM CET Nate Graham wrote: > > On 11/5/23 07:42, Kevin Ottens wrote: > > > I was clumsily advocating for this Akademy 2021 or 2022 (can't remember > > > which). > > > > > > This way it's clearer to application authors when they tie themselves > to a > > > given workspace or not. > > > > > > Also, isn't Elisa able to work without Baloo? It even seems to do the > > > right > > > thing if I trust its CMakeLists.txt. It has Baloo as a recommended but > > > optional dependency, and disable it altogether for Win32 builds. > > > > Yes, Elisa also includes an internal indexer, for use on Windows and > > Android, or on *Nix when Baloo isn't installed or is disabled. > > > > I think the original idea for the app was to delegate all the indexing > > heavy lifting to Baloo to avoid internal complication, but clearly this > > has not worked out in practice, since to be truly cross-platform, it > > can't assume that Baloo is present and active and does need its own > > indexer. So maybe the best course of action is actually to remove Baloo > > support entirely and always use the internal indexer, so that we don't > > have two different code paths. > > > > Nate > > I do not belive this is a good idea. Mostly because it means we would then > have an indexer in Baloo and one indexer in Elisa running at the same time > and > doing something similar which is wasteful. It becomes even more of an > issue > when other apps like Peruse or Arianna start doing the same and use their > own > indexer instead of Baloo on Plasma. > > This defeat the point of having a general purpose indexer in Plasma. > > Cheers, > Carl > > > >