<div dir="auto">Hi,<div dir="auto"><br></div><div dir="auto">+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.</div><div dir="auto"><br></div><div dir="auto">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.)</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Shantanu</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Nov 5, 2023, 11:53 PM Carl Schwan <<a href="mailto:carl@carlschwan.eu">carl@carlschwan.eu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sunday, November 5, 2023 6:01:38 PM CET Nate Graham wrote:<br>
> On 11/5/23 07:42, Kevin Ottens wrote:<br>
> > I was clumsily advocating for this Akademy 2021 or 2022 (can't remember<br>
> > which).<br>
> > <br>
> > This way it's clearer to application authors when they tie themselves to a<br>
> > given workspace or not.<br>
> > <br>
> > Also, isn't Elisa able to work without Baloo? It even seems to do the<br>
> > right<br>
> > thing if I trust its CMakeLists.txt. It has Baloo as a recommended but<br>
> > optional dependency, and disable it altogether for Win32 builds.<br>
> <br>
> Yes, Elisa also includes an internal indexer, for use on Windows and<br>
> Android, or on *Nix when Baloo isn't installed or is disabled.<br>
> <br>
> I think the original idea for the app was to delegate all the indexing<br>
> heavy lifting to Baloo to avoid internal complication, but clearly this<br>
> has not worked out in practice, since to be truly cross-platform, it<br>
> can't assume that Baloo is present and active and does need its own<br>
> indexer. So maybe the best course of action is actually to remove Baloo<br>
> support entirely and always use the internal indexer, so that we don't<br>
> have two different code paths.<br>
> <br>
> Nate<br>
<br>
I do not belive this is a good idea. Mostly because it means we would then <br>
have an indexer in Baloo and one indexer in Elisa running at the same time and <br>
doing something similar which is wasteful. It becomes even more of an issue <br>
when other apps like Peruse or Arianna start doing the same and use their own <br>
indexer instead of Baloo on Plasma.<br>
<br>
This defeat the point of having a general purpose indexer in Plasma.<br>
<br>
Cheers,<br>
Carl<br>
<br>
<br>
<br>
</blockquote></div>