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

Question

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

<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

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

我尝试通过管道将 stdout 和 stderr 传输到 /dev/null 以获取该命令，只是为了看看它是否有效，并且仍然打印出错误...

这完全是一个我可以解决的问题，它确实不会引起任何问题，但我不希望技术水平较低的同事看到这个错误并让它把它们扔掉(而且我不确定每次我们 merge 带有尾随空格的东西时是否会打印出来)，所以我很想知道发生了什么以及如何解决这个问题(或者告诉 git reset 忽略尾随空格错误)。

编辑:

这是一个完整的工作仓库，演示了这个问题:https://github.com/hjc1710/so-git-hook-question ，只需按照 README.md 中的步骤进行操作，无论您是否处于 merge 中，您都会收到错误。

可以在 here 找到我的 git 配置(我的工作站是 1.8.x，笔记本电脑是 2.5.x) .敏感信息已被删除，但这些信息都不应该是相关的。

Answer 1

作为 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，正如评论中指出的那样。

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

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