From 5b4de1c915e7f8db19bfe3907d1a301406a3ded7 Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Sat, 1 Dec 2018 16:50:40 +0100 Subject: [PATCH] Cygwin: mkvers: fix a bug in sed statement While reformatting the script, backticks `` were replaced with brackets $(). This in turn invalidated the \\( ... \\) expressions in the sed script because backslash resolution in $() works differently from backslash resolution in ``. Only a single backslash is valid now. While at it, fix up the uname(2) date representation when building a snapshot. Signed-off-by: Corinna Vinschen --- winsup/cygwin/mkvers.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/winsup/cygwin/mkvers.sh b/winsup/cygwin/mkvers.sh index 5aecb141b..6309c84b1 100755 --- a/winsup/cygwin/mkvers.sh +++ b/winsup/cygwin/mkvers.sh @@ -140,7 +140,8 @@ done | tee /tmp/mkvers.$$ 1>&9 trap "rm -f /tmp/mkvers.$$" 0 1 2 15 if [ -n "$snapshotdate" ]; then - usedate="$(echo $snapshotdate | sed 's/-\\(..:..[^-]*\\).*$/ \1SNP/')" + usedate="$(echo $snapshotdate \ + | sed -e 's/\(....\)\(..\)\(..\)-\(..:..\).*$/\1-\2-\3 \4SNP/')" else usedate="$builddate" fi