After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 610800 - At startup, Banshee should check for changes in the folder watched, when LibraryWatcher is turned on
At startup, Banshee should check for changes in the folder watched, when Libr...
Status: RESOLVED WONTFIX
Product: banshee
Classification: Other
Component: Other Extensions
git master
Other All
: Normal enhancement
: ---
Assigned To: Banshee Maintainers
Banshee Maintainers
gnome[unmaintained]
Depends on: 577225 638889
Blocks:
 
 
Reported: 2010-02-23 11:44 UTC by Praveen Thirukonda
Modified: 2020-03-17 08:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (4.74 KB, patch)
2013-01-13 14:49 UTC, Andrés G. Aragoneses (IRC: knocte)
none Details | Review
CPU spike (455.36 KB, image/png)
2013-01-16 22:54 UTC, Andrés G. Aragoneses (IRC: knocte)
  Details

Description Praveen Thirukonda 2010-02-23 11:44:03 UTC
many times i start banshe and folder watcher does nothing.
it should do a checkup once whne banshee starts.
Comment 1 mlaverdiere 2010-10-31 13:14:34 UTC
I'm experiencing the same problem (behavior).  Since folder watch extension will only be effective while Banshee is running, any change made to the library while Banshee is off will be missed.  To correct this, an automatic library update performed at Banshee launched should be implemented (maybe as an option, for those who don't want to have a startup time penalty).

Thanks.
Comment 2 Andrés G. Aragoneses (IRC: knocte) 2011-01-15 13:29:59 UTC
Confirming.

What I think we need for this bug is:

1) Get the patch of bug 638889 committed.
2) Taking advantage of the functionality implemented in bug 638889, create a patch here to do an automatic Import operation at startup if LibraryWatcher is on.
3) Create another patch to not only auto-import, but also auto-remove non-existant elements at startup. This patch should be done very carefully because there may be people that have music imported from external hard disks, and in case the hard disk is not mounted, the tracks shouldn't be removed.

I'm also removing the dependency to bug 610800 because this is not a bug in the LibraryWatcher, but an enhancement request.
Comment 3 Age Bosma (IRC: Forage) 2011-04-15 19:30:56 UTC
I would love to see this as well. It is quite confusing to have the library watcher enabled but nothing happening unless you make changed while Banshee is running. It is not the behaviour you would expect.

Fixing this issue would reduce the amount of issue reports. I've seen a few issues which are basically redundant after this is fixed.
Comment 4 Age Bosma (IRC: Forage) 2011-04-15 19:50:11 UTC
I forgot to mention that the issue got more severe with the close/quit changes in Banshee 2.0. Banshee will now quit more often instead of closing, thus not tracking library changes.
Comment 5 Andrés G. Aragoneses (IRC: knocte) 2011-12-14 16:48:30 UTC
(In reply to comment #2)
> I'm also removing the dependency to bug 610800 because this is not a bug in the
> LibraryWatcher, but an enhancement request.

For the record, in this comment, I meant bug 638939, not bug 610800 (which is this bug itself).
Comment 6 Andrés G. Aragoneses (IRC: knocte) 2011-12-14 18:52:52 UTC
BTW, I've marked bug 577225 as a dependency because if we implement this without that bug being fixed, we could make the user loose data.
Comment 7 Andrés G. Aragoneses (IRC: knocte) 2013-01-13 14:49:37 UTC
Created attachment 233385 [details] [review]
Proposed patch
Comment 8 Andrés G. Aragoneses (IRC: knocte) 2013-01-16 22:54:09 UTC
Created attachment 233634 [details]
CPU spike

With a library of 4,000 songs, a rescan takes about only 6 seconds in my laptop and this is the CPU spike experienced.

For the test, previous to the rescan operation and while Banshee was off, I removed an album, added a new one, and moved one to a different place.

IMHO despite the CPU spike, rescan is very fast and should be considered part of LibraryWatcher, since it's still opt-in.

Maybe before turning LibraryWatcher ON by default, what we have to consider is getting this fixed https://bugzilla.novell.com/show_bug.cgi?id=540524.

I, for one, will try to find time to get the BNC ticket above fixed.
Comment 9 Andrés G. Aragoneses (IRC: knocte) 2013-01-16 23:18:39 UTC
(In reply to comment #8)
> getting this fixed https://bugzilla.novell.com/show_bug.cgi?id=540524.
> 
> I, for one, will try to find time to get the BNC ticket above fixed.

Oops, sadly that would require root privileges to work.

:-m
Comment 10 Andrés G. Aragoneses (IRC: knocte) 2013-01-17 00:11:09 UTC
(In reply to comment #9)
> (In reply to comment #8)
> > getting this fixed https://bugzilla.novell.com/show_bug.cgi?id=540524.
> > 
> > I, for one, will try to find time to get the BNC ticket above fixed.
> 
> Oops, sadly that would require root privileges to work.
> 
> :-m

Yay, I also found this: http://stackoverflow.com/questions/14370050/nice-scheduling-priority-for-threads-within-a-process-instead-of-just-for-a-si
Comment 11 André Klapper 2020-03-17 08:52:13 UTC
Banshee is not under active development anymore and had its last code changes more than three years ago. Its codebase has been archived.

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect
reality. Please feel free to reopen this ticket (or rather transfer the project
to GNOME Gitlab, as GNOME Bugzilla is being shut down) if anyone takes the
responsibility for active development again.
See https://gitlab.gnome.org/Infrastructure/Infrastructure/issues/264 for more info.