git reset --hard -q 产生关于空格的错误

Question

我正在编写一个预提交钩子来运行我的 Python 测试，并且一切都运行良好......直到我遇到合并冲突。在此合并冲突中，引入了一些带有尾随空格的文档文件，每次我尝试提交时，都会收到以下消息：

<stdin>:28: trailing whitespace.
Ticket Link: 
<stdin>:54: trailing whitespace.
Then visit `http://app.dev` 
<stdin>:13528: trailing whitespace.
                            //most be provided for ALL resource updates, and 
<stdin>:13531: trailing whitespace.
    "deleted": false,  //indicates if this action resulted in a resource delete; 

warning: squelched 415 whitespace errors
warning: 420 lines add whitespace errors.
fatal: could not open '.git/MERGE_HEAD' for reading: No such file or directory

然后当我打开我的编辑器来写提交信息时，它完全是空的。

我的预提交脚本是：

#!/bin/sh

RED='\033[0;31m'
NC='\033[0m'

proper_pop() {
    git reset --hard -q 
    git stash apply -q --index && git stash drop -q
}

exit_and_pop() {
    proper_pop

    if [ "$1" -ne 0 ]; then
        echo "${RED}Your code failed the pre-commit hook! Please examine the output and fix your issues!${NC}"
    fi

    exit $1
}

run_and_bail() {
    bash -c "$1";
    ret=$?;

    if [ "${ret}" -ne 0 ]; then
        exit_and_pop "${ret}"
    fi
}

if git rev-parse --verify HEAD >/dev/null 2>&1
then
    against=HEAD
else
    # Initial commit: diff against an empty tree object
    against=4b825dc642cb6eb9a060e54bf8d69288fbee4904
fi

old_stash=$(git rev-parse -q --verify refs/stash)
git stash -q --keep-index
new_stash=$(git rev-parse -q --verify refs/stash)

if [ "$old_stash" = "$new_stash" ]; then
    echo "pre-commit script: No changes to test. Not running."
    sleep 1  # HACK: Editor may erase message if done too quickly, make the programmer read
    exit 0
fi

# If you want to allow non-ASCII filenames set this variable to true.
allownonascii=$(git config --bool hooks.allownonascii)

# Redirect output to stderr.
exec 1>&2

# Cross platform projects tend to avoid non-ASCII filenames; prevent
# them from being added to the repository. We exploit the fact that the
# printable range starts at the space character and ends with tilde.
if [ "$allownonascii" != "true" ] &&
    # Note that the use of brackets around a tr range is ok here, (it's
    # even required, for portability to Solaris 10's /usr/bin/tr), since
    # the square bracket bytes happen to fall in the designated range.
    test $(git diff --cached --name-only --diff-filter=A -z $against |
      LC_ALL=C tr -d '[ -~]\0' | wc -c) != 0
then
    cat <<\EOF
Error: Attempt to add a non-ASCII file name.
This can cause problems if you want to work with people on other platforms.
To be portable it is advisable to rename the file.
If you know what you are doing you can disable this check using:
  git config hooks.allownonascii true
EOF
    exit_and_pop 1
fi

if ! [ -z "$(which git-pylint-commit-hook)" ]; then
    run_and_bail "git-pylint-commit-hook"
fi

if ! [ -z "$(which pep8)" ]; then
    run_and_bail "python hooks/pep8-hook-check.py"
fi

proper_pop

通过我的调试，我已经生成了最小的脚本来重现这个错误：

#!/bin/sh

RED='\033[0;31m'
NC='\033[0m'

proper_pop() {
    git reset --hard -q 
    git stash apply -q --index && git stash drop -q
}

exit_and_pop() {
    proper_pop

    if [ "$1" -ne 0 ]; then
        echo "${RED}Your code failed the pre-commit hook! Please examine the output and fix your issues!${NC}"
    fi

    exit $1
}

run_and_bail() {
    bash -c "$1";
    ret=$?;

    if [ "${ret}" -ne 0 ]; then
        exit_and_pop "${ret}"
    fi
}

if git rev-parse --verify HEAD >/dev/null 2>&1
then
    against=HEAD
else
    # Initial commit: diff against an empty tree object
    against=4b825dc642cb6eb9a060e54bf8d69288fbee4904
fi

old_stash=$(git rev-parse -q --verify refs/stash)
git stash -q --keep-index
new_stash=$(git rev-parse -q --verify refs/stash)

if [ "$old_stash" = "$new_stash" ]; then
    echo "pre-commit script: No changes to test. Not running."
    sleep 1  # HACK: Editor may erase message if done too quickly, make the programmer read
    exit 0
fi

proper_pop

通过这个，我了解到这行：git reset --hard -q in proper_pop 是唯一的罪魁祸首（删除它会消除错误）。没有安装其他钩子，git 版本是：1.8.1.2，但我也在版本2.5.0 上运行了这个，同样的问题发生了。有没有人任何知道发生了什么？

我尝试将 stdout 和 stderr 传送到 /dev/null 以查看该命令是否有效，并且仍然打印错误...

这完全是一个我可以解决的问题，它确实不会导致任何问题，但我不希望技术水平较低的同事看到这个错误并让它摆脱它们（我不确定每次我们合并带有尾随空格的内容时是否会打印它），所以我很想知道发生了什么以及如何解决这个问题（或告诉git reset忽略尾随空格错误）。

编辑：

这是演示此问题的完整工作存储库：https://github.com/hjc1710/so-git-hook-question，只需按照README.md 中的步骤操作，无论您是否在合并中，您都会收到错误消息。

可以在here 找到我的 git 配置（适用于 1.8.x 上的工作站和 2.5.x 上的笔记本电脑）。敏感信息已被删除，但没有一个是相关的。

Answer 1

感谢您让这个复制变得如此容易！

问题似乎是钩子内的 git 命令正在将空白错误写入当前的tty 而不是stdout 或stderr。然后，hook runner 拾取错误，然后错误地声称它是stdout。

修复来自the answer 对我之前的一个问题。

#!/bin/sh
git stash > /dev/null
unbuffer sh -c 'git stash apply -q --index  && git stash drop -q ' >/dev/null 2>&1

编辑：如果你想要 OSX 上的 unbuffer 程序，你可以使用指令here 获得它。

brew tap homebrew/dupes/expect
brew install homebrew/dupes/expect

Answer 2

正如this answer 中的第二个脚注所指出的，git stash apply 只是调用git diff ... | git apply --index。警告来自这些命令。

当您有尾随空格、存储包含尾随空格的文件，然后应用存储时，就会发生这种情况。 更准确地说，每当您尝试将git apply 尾随换行符添加到代码体时，都会发生这种情况

如果您创建一个带有尾随空格的 git 补丁并尝试通过 cat a.patch | git diff --index 应用它，则会产生相同的错误。

至于解决这个问题，按理说在本地、全局或系统gitconfig 中设置core.whitespace=nowarn 会使其静音，但这对我来说似乎不是真的，但YMMV . （这些设置似乎没有被读取）。 运行git config core.whitespace -blank-at-eol，正如 cmets 中所指出的那样。

除了上述之外，我看到的唯一解决方案是在 git-stash bash 脚本中找到有问题的行，然后自己将 --whitespace=nowarn 参数添加到 git apply。我应该提到的是一个相当非标准、不可移植的解决方案。你必须让每个人都这样做，很可能每次 git 更新时也是如此。但是，如果他们无休止地打扰您，那就是您可以做的。

从好的方面来说，正如你所提到的，它可以解决，但如果我没有把事情弄得一团糟，那么在 git 上打开一个错误可能是值得的，而且它 甚至 忽略了系统 gitconfig。我希望它至少应该尊重这一点。