Skip to content

Commit 2fad78e

Browse files
committed
Fix a nasty typo
Upgrade of system libraries may expose this bug at any time and capital letters may be claimed not to be letters at all.
1 parent bb98cda commit 2fad78e

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

ChangeLog

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ Buxfixes:
55

66
* `#!/bin/bash` is no longer the shebang. The more portable `#!/usr/bin/env bash` is used instead (#185).
77
* Indentation is consistent in validation functions (#143)
8+
* Fixed bug preventing short options to have capital letters (#199)
89

910
New features:
1011

src/collectors.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ dnl Checks that the an argument is a correct short option arg
7373
dnl $1: The short option "string"
7474
dnl $2: The argument name
7575
m4_define([_CHECK_SHORT_OPTION_NAME_IS_OK], [m4_ifnblank([$1], [m4_do(
76-
[m4_bmatch([$1], [^[0-9a-zA-z?]$], ,
76+
[m4_bmatch([$1], [^[0-9a-zA-Z?]$], ,
7777
[_COLLECTOR_FEEDBACK([The value of short option '$1' for argument '--$2' is not valid - it has to be either left blank, or exactly one character.]m4_ifnblank([$1], [[ (Yours has ]m4_len([$1])[ characters).]]))])],
7878
[m4_set_contains([_ARGS_SHORT], [$1],
7979
[_COLLECTOR_FEEDBACK([The short option '$1' (in definition of '--$2') is already used.])],

0 commit comments

Comments
 (0)