Github messages for mblaze
 help / color / mirror / Atom feed
From: codesoap <codesoap@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [PR PATCH] [Updated] mcom: quote sequence for mbnc and mrep
Date: Fri, 11 Nov 2022 15:04:31 +0100	[thread overview]
Message-ID: <20221111140431.cftuLWz1J9GiZxIzVe6NO4BsHFhm8SvaClfoD-xAchg@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-fa6558a0-26e0-48f6-803f-f5a8af34f6a8-mblaze-231@inbox.vuxu.org>

[-- 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

  parent reply	other threads:[~2022-11-11 14:04 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-09 22:02 [PR PATCH] " codesoap
2022-11-09 23:43 ` leahneukirchen
2022-11-10  6:57 ` [PR PATCH] [Updated] " codesoap
2022-11-10  6:59 ` codesoap
2022-11-11 12:49 ` leahneukirchen
2022-11-11 14:04 ` codesoap [this message]
2022-11-11 14:06 ` codesoap
2022-11-11 21:27 ` [PR PATCH] [Closed]: " leahneukirchen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221111140431.cftuLWz1J9GiZxIzVe6NO4BsHFhm8SvaClfoD-xAchg@z \
    --to=codesoap@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).