[PATCH] notmuch-mutt: use --format=text0 and xargs -0

[PATCH] notmuch-mutt: use --format=text0 and xargs -0

[Jani Nikula]

notmuch-mutt fails for message files with special characters such as
single quote in their filename. Use notmuch search --format=text0 and
xargs -0 combo to handle them.

Reported and tested by "dob1" on IRC.
---
 contrib/notmuch-mutt/notmuch-mutt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/contrib/notmuch-mutt/notmuch-mutt b/contrib/notmuch-mutt/notmuch-mutt
index 0e46a8c1b95e..57f13075aa22 100755
--- a/contrib/notmuch-mutt/notmuch-mutt
+++ b/contrib/notmuch-mutt/notmuch-mutt
@@ -48,9 +48,9 @@ sub search($$$) {
     }
 
     empty_maildir($maildir);
-    system("notmuch search --output=files $dup_option $query"
+    system("notmuch search --format=text0 --output=files $dup_option $query"
 	   . " | sed -e 's: :\\\\ :g'"
-	   . " | xargs -r -I searchoutput ln -s searchoutput $maildir/cur/");
+	   . " | xargs -0 -r -I searchoutput ln -s searchoutput $maildir/cur/");
 }
 
 sub prompt($$) {
-- 
2.11.0

Re: [PATCH] notmuch-mutt: use --format=text0 and xargs -0

[Jani Nikula]

On Tue, 27 Feb 2018, Jani Nikula <jani@nikula.org> wrote:
> notmuch-mutt fails for message files with special characters such as
> single quote in their filename. Use notmuch search --format=text0 and
> xargs -0 combo to handle them.
>
> Reported and tested by "dob1" on IRC.
> ---
>  contrib/notmuch-mutt/notmuch-mutt | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/contrib/notmuch-mutt/notmuch-mutt b/contrib/notmuch-mutt/notmuch-mutt
> index 0e46a8c1b95e..57f13075aa22 100755
> --- a/contrib/notmuch-mutt/notmuch-mutt
> +++ b/contrib/notmuch-mutt/notmuch-mutt
> @@ -48,9 +48,9 @@ sub search($$$) {
>      }
>  
>      empty_maildir($maildir);
> -    system("notmuch search --output=files $dup_option $query"
> +    system("notmuch search --format=text0 --output=files $dup_option $query"
>  	   . " | sed -e 's: :\\\\ :g'"

Come to think of it, does this need sed -z too?

> -	   . " | xargs -r -I searchoutput ln -s searchoutput $maildir/cur/");
> +	   . " | xargs -0 -r -I searchoutput ln -s searchoutput $maildir/cur/");
>  }
>  
>  sub prompt($$) {
> -- 
> 2.11.0

Re: [PATCH] notmuch-mutt: use --format=text0 and xargs -0

[Tomi Ollila]

On Tue, Feb 27 2018, Jani Nikula wrote:

> On Tue, 27 Feb 2018, Jani Nikula <jani@nikula.org> wrote:
>> notmuch-mutt fails for message files with special characters such as
>> single quote in their filename. Use notmuch search --format=text0 and
>> xargs -0 combo to handle them.
>>
>> Reported and tested by "dob1" on IRC.
>> ---
>>  contrib/notmuch-mutt/notmuch-mutt | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/contrib/notmuch-mutt/notmuch-mutt b/contrib/notmuch-mutt/notmuch-mutt
>> index 0e46a8c1b95e..57f13075aa22 100755
>> --- a/contrib/notmuch-mutt/notmuch-mutt
>> +++ b/contrib/notmuch-mutt/notmuch-mutt
>> @@ -48,9 +48,9 @@ sub search($$$) {
>>      }
>>  
>>      empty_maildir($maildir);
>> -    system("notmuch search --output=files $dup_option $query"
>> +    system("notmuch search --format=text0 --output=files $dup_option $query"
>>  	   . " | sed -e 's: :\\\\ :g'"
>
> Come to think of it, does this need sed -z too?

it looks  to me that sed can  be dropped.

btw how many non-coreutils xargs support -0 option ?




>
>> -	   . " | xargs -r -I searchoutput ln -s searchoutput $maildir/cur/");
>> +	   . " | xargs -0 -r -I searchoutput ln -s searchoutput $maildir/cur/");
>>  }
>>  
>>  sub prompt($$) {
>> -- 
>> 2.11.0
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

Re: [PATCH] notmuch-mutt: use --format=text0 and xargs -0

[Tomi Ollila]

On Wed, Feb 28 2018, Tomi Ollila wrote:

> On Tue, Feb 27 2018, Jani Nikula wrote:
>
>> On Tue, 27 Feb 2018, Jani Nikula <jani@nikula.org> wrote:
>>> notmuch-mutt fails for message files with special characters such as
>>> single quote in their filename. Use notmuch search --format=text0 and
>>> xargs -0 combo to handle them.
>>>
>>> Reported and tested by "dob1" on IRC.
>>> ---
>>>  contrib/notmuch-mutt/notmuch-mutt | 4 ++--
>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/contrib/notmuch-mutt/notmuch-mutt b/contrib/notmuch-mutt/notmuch-mutt
>>> index 0e46a8c1b95e..57f13075aa22 100755
>>> --- a/contrib/notmuch-mutt/notmuch-mutt
>>> +++ b/contrib/notmuch-mutt/notmuch-mutt
>>> @@ -48,9 +48,9 @@ sub search($$$) {
>>>      }
>>>  
>>>      empty_maildir($maildir);
>>> -    system("notmuch search --output=files $dup_option $query"
>>> +    system("notmuch search --format=text0 --output=files $dup_option $query"
>>>  	   . " | sed -e 's: :\\\\ :g'"
>>
>> Come to think of it, does this need sed -z too?
>
> it looks  to me that sed can  be dropped.
>
> btw how many non-coreutils xargs support -0 option ?

Checked (briefly):

openbsd, netbsd, macos and solaris xargs(1) (latest versions...) know -0 option.

so

    system("notmuch search --format=text0 --output=files $dup_option $query"
	   . " | xargs -0 -r -I searchoutput ln -s searchoutput $maildir/cur/");

should be supported enough.


Tomi