Re: [PR PATCH] [Updated] mcom: do not match shebang in needs_multipart()

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

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

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

https://github.com/codesoap/mblaze shebang
https://github.com/leahneukirchen/mblaze/pull/196

mcom: do not match shebang in needs_multipart()
Avoids matching lines like `#!/usr/bin/env sh`. Also require at least one character after `/`.

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

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

From e37d2102e32977e4f17fb9c560fb0bcd6743b4be Mon Sep 17 00:00:00 2001
From: codesoap <codesoap@mailbox.org>
Date: Mon, 4 Jan 2021 18:55:48 +0100
Subject: [PATCH] mcom: do not match shebang in needs_multipart()

Avoids matching lines like '#!/usr/bin/env sh'.
---
 mcom | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mcom b/mcom
index f0de75c..c378b33 100755
--- a/mcom
+++ b/mcom
@@ -75,7 +75,7 @@ stripempty() {
 
 needs_multipart() {
 	mhdr -h attach "$1" >/dev/null ||
-		grep -q '^#[^ ]*/[^ ]* ' "$1"
+		grep -qE '^#[a-zA-Z]+/[a-zA-Z0-9+.;=#-]+ ' "$1"
 }
 
 do_mime() {