Re: [PATCH] Expand docstrings about fcc using notmuch-insert
Keith Amidon writes: > This commit expands docstrings for notmuch-fcc-dirs and > notmuch-maildir-fcc-with-notmuch-insert to describe how quoted strings > are processed and make the ability to configure sent folders containing > whitespace more discoverable. pushed to release and master d ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] Expand docstrings about fcc using notmuch-insert
Hi This version look good to me +1. I recommend it for 0.23.1. Best wishes Mark On Wed, 19 Oct 2016, Keith Amidon wrote: > This commit expands docstrings for notmuch-fcc-dirs and > notmuch-maildir-fcc-with-notmuch-insert to describe how quoted strings > are processed and make the ability to configure sent folders containing > whitespace more discoverable. > --- > emacs/notmuch-maildir-fcc.el | 13 ++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/emacs/notmuch-maildir-fcc.el b/emacs/notmuch-maildir-fcc.el > index 95e5650..ea75bb9 100644 > --- a/emacs/notmuch-maildir-fcc.el > +++ b/emacs/notmuch-maildir-fcc.el > @@ -54,7 +54,10 @@ If `notmuch-maildir-use-notmuch-insert' is set (the > default) then > the header should be of the form \"folder +tag1 -tag2\" where > folder is the folder (relative to the notmuch mailstore) to store > the message in, and tag1 and tag2 are tag changes to apply to the > -stored message. > +stored message. This string is split using `split-string-and-unquote', > +so a folder name containing spaces can be specified by > +quoting each space with an immediately preceding backslash > +or surrounding the entire folder name in double quotes. > > If `notmuch-maildir-use-notmuch-insert' is nil then the Fcc > header should be the directory where the message should be > @@ -230,8 +233,12 @@ should be a list of tag changes to apply to the inserted > message." > The fcc-header should be of the form \"folder +tag1 -tag2\" where > folder is the folder (relative to the notmuch mailstore) to store > the message in, and tag1 and tag2 are tag changes to apply to the > -stored message. If CREATE is non-nil then create the folder if > -necessary." > +stored message. This string is split using `split-string-and-unquote', > +so a folder name containing spaces can be specified by > +quoting each space with an immediately preceding backslash > +or surrounding the entire folder name in double quotes. > + > +If CREATE is non-nil then create the folder if necessary." >(let* ((args (split-string-and-unquote fcc-header)) >(folder (car args)) >(tags (cdr args))) > -- > 2.10.0 > > ___ > notmuch mailing list > notmuch@notmuchmail.org > https://notmuchmail.org/mailman/listinfo/notmuch ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
[PATCH] Expand docstrings about fcc using notmuch-insert
This commit expands docstrings for notmuch-fcc-dirs and notmuch-maildir-fcc-with-notmuch-insert to describe how quoted strings are processed and make the ability to configure sent folders containing whitespace more discoverable. --- emacs/notmuch-maildir-fcc.el | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/emacs/notmuch-maildir-fcc.el b/emacs/notmuch-maildir-fcc.el index 95e5650..ea75bb9 100644 --- a/emacs/notmuch-maildir-fcc.el +++ b/emacs/notmuch-maildir-fcc.el @@ -54,7 +54,10 @@ If `notmuch-maildir-use-notmuch-insert' is set (the default) then the header should be of the form \"folder +tag1 -tag2\" where folder is the folder (relative to the notmuch mailstore) to store the message in, and tag1 and tag2 are tag changes to apply to the -stored message. +stored message. This string is split using `split-string-and-unquote', +so a folder name containing spaces can be specified by +quoting each space with an immediately preceding backslash +or surrounding the entire folder name in double quotes. If `notmuch-maildir-use-notmuch-insert' is nil then the Fcc header should be the directory where the message should be @@ -230,8 +233,12 @@ should be a list of tag changes to apply to the inserted message." The fcc-header should be of the form \"folder +tag1 -tag2\" where folder is the folder (relative to the notmuch mailstore) to store the message in, and tag1 and tag2 are tag changes to apply to the -stored message. If CREATE is non-nil then create the folder if -necessary." +stored message. This string is split using `split-string-and-unquote', +so a folder name containing spaces can be specified by +quoting each space with an immediately preceding backslash +or surrounding the entire folder name in double quotes. + +If CREATE is non-nil then create the folder if necessary." (let* ((args (split-string-and-unquote fcc-header)) (folder (car args)) (tags (cdr args))) -- 2.10.0 ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] Expand docstrings about fcc using notmuch-insert
Hi Many thanks for doing this (and bringing this to out attention). I have a couple of minor suggestions for tweaking what you have written, but I am not certain they are improvements, so comments gratefully received. On Tue, 11 Oct 2016, Keith Amidon wrote: > This commit expands docstrings for notmuch-fcc-dirs and > notmuch-maildir-fcc-with-notmuch-insert to describe how quoted strings > are processed and make the ability to configure sent folders containing > whitespace more discoverable. > --- > emacs/notmuch-maildir-fcc.el | 13 ++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/emacs/notmuch-maildir-fcc.el b/emacs/notmuch-maildir-fcc.el > index 95e5650..cfe1931 100644 > --- a/emacs/notmuch-maildir-fcc.el > +++ b/emacs/notmuch-maildir-fcc.el > @@ -54,7 +54,10 @@ If `notmuch-maildir-use-notmuch-insert' is set (the > default) then > the header should be of the form \"folder +tag1 -tag2\" where > folder is the folder (relative to the notmuch mailstore) to store > the message in, and tag1 and tag2 are tag changes to apply to the > -stored message. > +stored message. This string is split using `split-string-and-unquote', > +so a folder name containing whitespace can be specified by > +quoting the whitespace with an immediately preceeding backslash > +or surrounding the entire folder name in double quotes. > > If `notmuch-maildir-use-notmuch-insert' is nil then the Fcc > header should be the directory where the message should be > @@ -230,8 +233,12 @@ should be a list of tag changes to apply to the inserted > message." > The fcc-header should be of the form \"folder +tag1 -tag2\" where > folder is the folder (relative to the notmuch mailstore) to store > the message in, and tag1 and tag2 are tag changes to apply to the > -stored message. If CREATE is non-nil then create the folder if > -necessary." > +stored message. This string is split using `split-string-and-unquote', > +so a folder name containing whitespace can be specified by quoting > +the whitespace with an immediately preceeding backslash or > +surrounding the entire folder name in double quotes. Maybe "...containing whitespace can be specified by quoting each whitespace character with an immediately preceding backslash ..." Or maybe "...so a folder name containing spaces can be specified by quoting each space with an immediately preceding backslash ..." The former is more accurate but the latter is simpler, and I don't think tabs and newlines in filenames occur often enough to care about (in fact I am not sure we cope with a newline in an fcc header anyway) Best wishes Mark > + > +If CREATE is non-nil then create the folder if necessary." >(let* ((args (split-string-and-unquote fcc-header)) >(folder (car args)) >(tags (cdr args))) > -- > 2.10.0 > > ___ > notmuch mailing list > notmuch@notmuchmail.org > https://notmuchmail.org/mailman/listinfo/notmuch ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] Expand docstrings about fcc using notmuch-insert
On Thu, 2016-10-13 at 22:30 -0300, David Bremner wrote: > No technical difficulties, we're just not as good as you think we are > at quick reviews ;). I wanted to hear what Mark felt about exposing > these internals as docstring. I OK with it, but it does make it > slightly harder to change things later. Thanks for getting back to me. In case it helps, part of the reason why I mentioned exactly how it was done was because I wasn't 100% sure I captured the behavior of split-string-and-unquote correctly. I personally found the docstring, elisp manual page, and the source for that function all somewhat opaque. This way if someone ran into a problem trying to quote the folder as I described they would know where to go look for more detail. :-) --- Keith ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] Expand docstrings about fcc using notmuch-insert
Keith Amidon writes: > Did my attempt to submit via git send-email the documentation patch > quoted below somehow not get forwarded by the list server? I never got > a copy forwarded by the list but just assumed that it was suppressing > copies to the authors. Having observed that folks here are usually > very good about reviewing patch submissions quickly and not having > gotten any feedback, I'm wondering if something went wrong after all... > > --- Keith Hi Keith; No technical difficulties, we're just not as good as you think we are at quick reviews ;). I wanted to hear what Mark felt about exposing these internals as docstring. I OK with it, but it does make it slightly harder to change things later. d ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] Expand docstrings about fcc using notmuch-insert
Did my attempt to submit via git send-email the documentation patch quoted below somehow not get forwarded by the list server? I never got a copy forwarded by the list but just assumed that it was suppressing copies to the authors. Having observed that folks here are usually very good about reviewing patch submissions quickly and not having gotten any feedback, I'm wondering if something went wrong after all... --- Keith On Mon, 2016-10-10 at 21:52 -0700, Keith Amidon wrote: > This commit expands docstrings for notmuch-fcc-dirs and > notmuch-maildir-fcc-with-notmuch-insert to describe how quoted > strings > are processed and make the ability to configure sent folders > containing > whitespace more discoverable. > --- > emacs/notmuch-maildir-fcc.el | 13 ++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/emacs/notmuch-maildir-fcc.el b/emacs/notmuch-maildir- > fcc.el > index 95e5650..cfe1931 100644 > --- a/emacs/notmuch-maildir-fcc.el > +++ b/emacs/notmuch-maildir-fcc.el > @@ -54,7 +54,10 @@ If `notmuch-maildir-use-notmuch-insert' is set > (the default) then > the header should be of the form \"folder +tag1 -tag2\" where > folder is the folder (relative to the notmuch mailstore) to store > the message in, and tag1 and tag2 are tag changes to apply to the > -stored message. > +stored message. This string is split using `split-string-and- > unquote', > +so a folder name containing whitespace can be specified by > +quoting the whitespace with an immediately preceeding backslash > +or surrounding the entire folder name in double quotes. > > If `notmuch-maildir-use-notmuch-insert' is nil then the Fcc > header should be the directory where the message should be > @@ -230,8 +233,12 @@ should be a list of tag changes to apply to the > inserted message." > The fcc-header should be of the form \"folder +tag1 -tag2\" where > folder is the folder (relative to the notmuch mailstore) to store > the message in, and tag1 and tag2 are tag changes to apply to the > -stored message. If CREATE is non-nil then create the folder if > -necessary." > +stored message. This string is split using `split-string-and- > unquote', > +so a folder name containing whitespace can be specified by quoting > +the whitespace with an immediately preceeding backslash or > +surrounding the entire folder name in double quotes. > + > +If CREATE is non-nil then create the folder if necessary." > (let* ((args (split-string-and-unquote fcc-header)) > (folder (car args)) > (tags (cdr args))) ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
[PATCH] Expand docstrings about fcc using notmuch-insert
This commit expands docstrings for notmuch-fcc-dirs and notmuch-maildir-fcc-with-notmuch-insert to describe how quoted strings are processed and make the ability to configure sent folders containing whitespace more discoverable. --- emacs/notmuch-maildir-fcc.el | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/emacs/notmuch-maildir-fcc.el b/emacs/notmuch-maildir-fcc.el index 95e5650..cfe1931 100644 --- a/emacs/notmuch-maildir-fcc.el +++ b/emacs/notmuch-maildir-fcc.el @@ -54,7 +54,10 @@ If `notmuch-maildir-use-notmuch-insert' is set (the default) then the header should be of the form \"folder +tag1 -tag2\" where folder is the folder (relative to the notmuch mailstore) to store the message in, and tag1 and tag2 are tag changes to apply to the -stored message. +stored message. This string is split using `split-string-and-unquote', +so a folder name containing whitespace can be specified by +quoting the whitespace with an immediately preceeding backslash +or surrounding the entire folder name in double quotes. If `notmuch-maildir-use-notmuch-insert' is nil then the Fcc header should be the directory where the message should be @@ -230,8 +233,12 @@ should be a list of tag changes to apply to the inserted message." The fcc-header should be of the form \"folder +tag1 -tag2\" where folder is the folder (relative to the notmuch mailstore) to store the message in, and tag1 and tag2 are tag changes to apply to the -stored message. If CREATE is non-nil then create the folder if -necessary." +stored message. This string is split using `split-string-and-unquote', +so a folder name containing whitespace can be specified by quoting +the whitespace with an immediately preceeding backslash or +surrounding the entire folder name in double quotes. + +If CREATE is non-nil then create the folder if necessary." (let* ((args (split-string-and-unquote fcc-header)) (folder (car args)) (tags (cdr args))) -- 2.10.0 ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
