Bug 387437 - Elisa unaware of Baloo if it starts after Elisa
Summary: Elisa unaware of Baloo if it starts after Elisa
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: baloo (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-29 13:16 UTC by Alexander Potashev
Modified: 2018-09-13 19:50 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Potashev 2017-11-29 13:16:52 UTC
1. Clear Elisa local data so that it has no information about local files.
2. Stop Baloo.
3. Start Elisa.
4. Start Baloo.
5. Open e.g. the Albums view in Elisa - it doesn't detect the local files (see also bug #387434 ). It should be able to detect local files because Baloo is already running.
Comment 1 Matthieu Gallien 2017-11-29 13:25:20 UTC
Yes, I know that problem exists. I have work in progress monitoring of Baloo.
Comment 2 Matthieu Gallien 2017-12-10 09:57:18 UTC
Hello,
I would like to work on your report but I am not sure of exactly what you mean by stop Baloo.
When working on the Baloo support, I am doing two different things to stop Baloo. The first is to deactivate file indexing (by example by using the KCM module). The second one is to stop the indexer (by example with balooctl stop). There is also a third one that is asking Baloo to only perform file indexing but without indexing content.
Thanks in advance
Best regards
Comment 3 Alexander Potashev 2017-12-10 13:52:49 UTC
In the step "Stop Baloo" I did the following:
 1. Uncheck "Enable File Search" (the topmost checkbox) in the Baloo KCM,
 2. Killed all processes that have "baloo" in their name (baloo_file, baloo_file_extractor, ...).
Comment 4 Matthieu Gallien 2018-09-03 20:29:49 UTC
Sorry for the delay. I can reproduce your problem. I am working on it.
Comment 5 Matthieu Gallien 2018-09-05 15:40:10 UTC
I have created a phabricator review for a fix (https://phabricator.kde.org/D15298)
Comment 6 Matthieu Gallien 2018-09-13 19:50:07 UTC
Git commit 77d5946c660ff87237a4b460a7f3de19e42167b2 by Matthieu Gallien.
Committed on 13/09/2018 at 19:49.
Pushed by mgallien into branch '0.3'.

fix handling of baloo starting after elisa

Summary:
if Baloo is configured inactive at elisa start and later is enabled,
fix baloo management such that we correctly query Baloo for a list of
audio files

Test Plan: manual test allow to get the music files from baloo

Reviewers: astippich, januz, aspotashev

Reviewed By: astippich

Tags: #elisa

Differential Revision: https://phabricator.kde.org/D15298

M  +4    -0    src/CMakeLists.txt
M  +61   -21   src/baloo/localbaloofilelisting.cpp
M  +2    -0    src/baloo/localbaloofilelisting.h

https://commits.kde.org/elisa/77d5946c660ff87237a4b460a7f3de19e42167b2