sed 命令卡住： sed -n "w $tmpfile" -- st-copyout 脚本

Question

在以下脚本中（表单here）

#!/bin/sh
# Using external pipe with st, give a dmenu prompt of recent commands,
# allowing the user to copy the output of one.
# xclip required for this script.
# By Jaywalker and Luke
tmpfile=$(mktemp /tmp/st-cmd-output.XXXXXX)
trap 'rm "$tmpfile"' 0 1 15
sed -n "w $tmpfile"
sed -i 's/\x0//g' "$tmpfile"
ps1="$(grep "\S" "$tmpfile" | tail -n 1 | sed 's/^\s*//' | cut -d' ' -f1)"
chosen="$(grep -F "$ps1" "$tmpfile" | sed '$ d' | tac | dmenu -p "Copy which command's output?" -i -l 10 | sed 's/[^^]/[&]/g; s/\^/\\^/g')"
eps1="$(echo "$ps1" | sed 's/[^^]/[&]/g; s/\^/\\^/g')"
awk "/^$chosen$/{p=1;print;next} p&&/$eps1/{p=0};p" "$tmpfile" | xclip -selection clipboard

为什么脚本卡在sed -n "w $tmpfile"

我在sed 手册页中查找了w 命令，上面没有模式也没有输入文件。

它曾经在我的机器上工作过！但不确定发生了什么以及导致它变成这样的原因......

Answer 1

使用sed -n "w $tmpfile"; sed -i 's/\x0//g' "$tmpfile"，您将管道输入复制到临时文件，然后使用带有“就地”编辑选项的 sed 来修改该临时文件。与只调用 1 次 sed 相比，这没有任何意义：sed 's/\x0//g' > "$tmpfile"

至于为什么挂起：

1. 猜测 1：您没有通过管道输入任何输入（第一条评论说 # Using external pipe... 从而告诉您即使您看不懂脚本，该命令也需要管道输入）。
2. 猜想 2：你现在的机器上的 sed 版本需要备份文件名（例如 BSD sed），所以 sed -i 's/\x0//g' "$tmpfile" 使用 's/\x0//g' 作为备份文件名，"$tmpfile" 作为备份文件名没有输入的脚本。

该命令还有其他几个可移植性、稳健性和效率问题 - 您可能希望将其丢弃并编写不同的脚本。

我又看了一下那个脚本，你绝对应该把它扔掉，因为它有很多错误，例如一行一行地在不知道输入/输出是什么的情况下如此猜测：

#!/bin/sh

• 应该是#!/usr/bin/env bash

tmpfile=$(mktemp /tmp/st-cmd-output.XXXXXX)

• 应该是tmpfile=$(mktemp /tmp/st-cmd-output.XXXXXX) || exit

trap 'rm "$tmpfile"' 0 1 15

• 应该是trap 'rm -f "$tmpfile"; exit' 0 1 15，而 1 和 15 可能不是必需的。

sed -n "w $tmpfile"
sed -i 's/\x0//g' "$tmpfile"

• 应该只是 1 个命令，sed 's/\x0//g' > "$tmpfile"。

ps1="$(grep "\S" "$tmpfile" | tail -n 1 | sed 's/^\s*//' | cut -d' ' -f1)"

• 应该是ps1=$(awk 'NF{line=$1} END{print line}' "$tmpfile")

chosen="$(grep -F "$ps1" "$tmpfile" | sed '$ d' | tac | dmenu -p "Copy which command's output?" -i -l 10 | sed 's/[^^]/[&]/g; s/\^/\\^/g')"

• 应该是chosen=$(tac "$tmpfile" | awk -v ps1="$ps1" 'index($0,ps1) && c++' | dmenu -p "Copy which command's output?" -i -l 10)

eps1="$(echo "$ps1" | sed 's/[^^]/[&]/g; s/\^/\\^/g')"
awk "/^$chosen$/{p=1;print;next} p&&/$eps1/{p=0};p" "$tmpfile" | xclip -selection clipboard

• 应该只是 1 个命令 awk -v chosen="$chosen" -v ps1="$ps1" '$0==chosen{p=1;print;next} p&&index($0,ps1){p=0};p' "$tmpfile" | xclip -selection clipboard

因此，总体而言，如果将脚本编写为：

#!/usr/bin/env bash
tmpfile=$(mktemp /tmp/st-cmd-output.XXXXXX) || exit

trap 'rm -f "$tmpfile"; exit' 0 1 15

sed 's/\x0//g' > "$tmpfile"

ps1=$(awk 'NF{line=$1} END{print line}' "$tmpfile")

chosen=$(tac "$tmpfile" | awk -v ps1="$ps1" 'index($0,ps1) && c++' | dmenu -p "Copy which command's output?" -i -l 10)

awk -v chosen="$chosen" -v ps1="$ps1" '$0==chosen{p=1;print;next} p&&index($0,ps1){p=0};p' "$tmpfile" | xclip -selection clipboard

原始脚本中的那些 sed 's/[^^]/[&]/g; s/\^/\\^/g' 命令试图转义字符串中的所有字符，以使任何正则表达式元字符在 awk 中的正则表达式中使用时都被视为文字，但它们的替换不正确（请参阅 Is it possible to escape regex metacharacters reliably with sed ) 如果您只是在 awk 中使用字符串而不是正则表达式，则无论如何都没有必要。

由于没有任何东西可以测试它，我可能有一些非常轻微的错误，并且可能会有进一步的改进，但我提供的命令将更接近于可移植、健壮、高效的脚本。