-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Wishlist: Command to update local tag cache #5
Comments
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. |
Great news. I shall therefore give up on trying to get an Apache server to run mopidy local scan successfully :) |
@adamcik how is this coming along? I'd love to help if there's anything I can do. |
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. |
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. |
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. |
Is this already added? |
Don't think so. |
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.
The text was updated successfully, but these errors were encountered: