Re: [PR PATCH] [Updated] mcom: quote sequence for mbnc and mrep

[codesoap <codesoap@users.noreply.github.com>]

[-- Attachment #1: Type: text/plain, Size: 590 bytes --]

There is an updated pull request by codesoap against master on the mblaze repository

https://github.com/codesoap/mblaze master
https://github.com/leahneukirchen/mblaze/pull/231

mcom: quote sequence for mbnc and mrep
I was getting the "mrep: needs exactly one mail to reply to" error when trying to reply to messages, where the file was located in a directory with spaces. With this change, I was able to use `mrep` in such a case. I noticed the same code in `mbnc`, so I also applied the change there.

A patch file from https://github.com/leahneukirchen/mblaze/pull/231.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-master-231.patch --]
[-- Type: text/x-diff, Size: 780 bytes --]

From 21a8308eac20e2506938d4b85ad47fd058933e98 Mon Sep 17 00:00:00 2001
From: codesoap <codesoap@mailbox.org>
Date: Wed, 9 Nov 2022 22:55:18 +0100
Subject: [PATCH] mcom: allow spaces in sequence for mbnc and mrep

---
 mcom | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/mcom b/mcom
index d60070e..fb48401 100755
--- a/mcom
+++ b/mcom
@@ -321,7 +321,10 @@ fi
 		) fi
 		;;
 	*mbnc*)
+		old_ifs="$IFS"
+		IFS=$NL
 		set -- $(mseq -- "$@")
+		IFS="$old_ifs"
 		if [ "$#" -ne 1 ]; then
 			printf 'mbnc: needs exactly one mail to bounce\n' 1>&2
 			exit 1
@@ -344,7 +347,10 @@ fi
 		)
 		;;
 	*mrep*)
+		old_ifs="$IFS"
+		IFS=$NL
 		set -- $(mseq -- "$@")
+		IFS="$old_ifs"
 		if [ "$#" -ne 1 ]; then
 			printf 'mrep: needs exactly one mail to reply to\n' 1>&2
 			exit 1