-
Notifications
You must be signed in to change notification settings - Fork 98
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
Configuration is not took in account when using --enable-filters #40
Comments
Yes, that's a problem. FWIW I consider just dropping the -e command line switch b/c I do not believe that it provides a real benefit. Do you see a reason to keep it and fix the behavior? |
On Mon 17 Sep 2012 - 04:32:57 [-0700], Justus Winter wrote:
I don't mind. I found it convenient but then lost quite a long time figuring A compromise would be to have a switch to filter out some filters. In this Nicolas Pouillard |
My use case would be: Alternative approach: patch notmuch... |
For example, I'm writing a filer to snooze mails, and it had to work on old mail. So my notmuch
|
I ran into the same issue. Please fix it, it also took me quite some time to realise that this is indeed a bug and not me. |
Update: I have a patch for notmuch that tags renamed (moved) messages, I'll try to upstream it shortly. This would support the mentioned use case: run |
Agreed with @matclab, ability to run single filter without relaying on config file is useful. Also relates to #237, in the way that, if it would be possible to run |
Filter classes must be initialized with the options found in the config file, but in the case of `--enable-filters` classes were being initialized and config options ignored. This change makes use of the already present `configured_filter_chain` list of already-initialized filters and trims it down by comparing filter class names with the values passed in the command line. refs afewmail#40
Filter objects must be initialized with the options found in the config file but, in the case of `--enable-filters`, new objects were being initialized with config options ignored. This change makes use of the `configured_filter_chain` list of already-initialized filter objects and trims it down by comparing filter class names with the values passed in the command line for `--enable-filters`. Closes: afewmail#40
For instance when using:
I found no way to set the sent_tag option. In particular putting it in the config was useless.
The text was updated successfully, but these errors were encountered: