mopidy-local icon indicating copy to clipboard operation
mopidy-local copied to clipboard

Wishlist: Command to update local tag cache

Open fatg3erman opened this issue 11 years ago • 8 comments
trafficstars

Now that mopidy_scan is no more and that functionality is integrated into mopidy, would it be possible to implement mpd's 'update' command to permit the local files cache to be updated by a client. Many people are running mopidy on headless servers with remote clients, and running 'mopidy local scan' from a client isn't elegant or even possible sometimes.

The mopidy.library.refresh() API command would seem to be the obvious place to add this.

fatg3erman avatar Dec 16 '13 10:12 fatg3erman

We are fairly close to being able to add this. I just want to get my registry work in first, given that I'm improving how we extend the local library.

adamcik avatar Dec 16 '13 10:12 adamcik

Great news. I shall therefore give up on trying to get an Apache server to run mopidy local scan successfully :)

fatg3erman avatar Dec 16 '13 10:12 fatg3erman

@adamcik how is this coming along? I'd love to help if there's anything I can do.

kcrwfrd avatar Apr 11 '14 04:04 kcrwfrd

Wiring up the update command to refresh should be trivial, this will not trigger a rescan, but it will ensure that the results from 'mopidy local scan' get loaded into memory.

adamcik avatar Oct 19 '14 12:10 adamcik

As far as I can tell, the HTTP API already does load the results of mopidy local scan via mopidy.library.refresh().

But I think a way of triggering a rescan - or even better would be automatic monitoring of the local files, with some HTTP API message to tell a client when something has changed - is a needed feature.

fatg3erman avatar Oct 19 '14 14:10 fatg3erman

Mhm, I agree about that, but even just wiring it up without the re-scan is a lot better than nothing.

mopidy/mopidy#742 is part of my plan to add background scanning using inotify with a mtime based fallback.

adamcik avatar Oct 19 '14 15:10 adamcik

Is this already added?

StephanMeijer avatar Feb 07 '16 15:02 StephanMeijer

Don't think so.

adamcik avatar Feb 07 '16 15:02 adamcik