parsemail.sh: Fix usage via stdin.

Message ID 20170314180912.GA29238@linux-mips.org
State Accepted
Headers show

Commit Message

Ralf Baechle March 14, 2017, 6:09 p.m.
Commit e56391f66cb8 ("trivial: Standardize variable naming in scripts")
sneaked in a checked for missing command line arguments in parsemail.sh
which broke typical argument-less usage for mail delivery.  Revert that
again.

Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
---
 patchwork/bin/parsemail.sh | 5 -----
 1 file changed, 5 deletions(-)

Comments

Stephen Finucane March 21, 2017, 8:35 p.m. | #1
On Tue, 2017-03-14 at 19:09 +0100, Ralf Baechle wrote:
> Commit e56391f66cb8 ("trivial: Standardize variable naming in
> scripts")
> sneaked in a checked for missing command line arguments in
> parsemail.sh
> which broke typical argument-less usage for mail delivery.  Revert
> that
> again.
> 
> Signed-off-by: Ralf Baechle <ralf@linux-mips.org>

Reviewed-by: Stephen Finucane <stephen@that.guru>

...and applied.

Patch

diff --git a/patchwork/bin/parsemail.sh b/patchwork/bin/parsemail.sh
index 5d1672e..a742f60 100755
--- a/patchwork/bin/parsemail.sh
+++ b/patchwork/bin/parsemail.sh
@@ -22,11 +22,6 @@ 
 BIN_DIR=$(dirname "$0")
 PATCHWORK_BASE=$(readlink -e "$BIN_DIR/../..")
 
-if [ $# -lt 1 ]; then
-    echo "usage: $0 <file> [options]" >&2
-    exit 1
-fi
-
 if [ -z "$PW_PYTHON" ]; then
     PW_PYTHON=python2
 fi