-
Notifications
You must be signed in to change notification settings - Fork 43
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
shelling out to "mu find" is brittle and should be replaced #5
Comments
Using the mu daemon would be much nicer rather than calling mu through shell. I've no real idea on how to do that though. |
(Disclaimer: I'm the author of I had to use shell commands when I wrote mu4e-maildirs-extension and it gave me lots of troubles. AFAIK it is now possible to fetch unread/total counts via the daemon. Hope this helps, although I haven't followed the mu development so I may be wrong. |
I've raised djcb/mu#1845 to see what @djcb might have to say about it. |
Thanks. Let's wait for an answer. |
I'm finding a bunch of my queries are not updating because of escaping issues when calling (shell-command-to-string) with my default shell (fish). At a minimum it would probably make sense to factor out all the queries to a single common function where we can be sure of getting the escaping right. However going forward we should investigate if @djcb could consider an public API to make queries of the "mu" daemon which already has the database loaded in memory and could probably just count the number of hits for you and respond with that.
The text was updated successfully, but these errors were encountered: