假设git status
给出了这样的:
# On branch X
# Changes to be committed:
# (use "git reset HEAD <file>..." to unstage)
#
# modified: file1.cc
# modified: file1.h
# modified: file1_test.cc
# modified: SConscript
#
# Changes not staged for commit:
# (use "git add <file>..." to update what will be committed)
# (use "git checkout -- <file>..." to discard changes in working directory)
# (commit or discard the untracked or modified content in submodules)
#
# modified: file1.cc
# modified: tinyxml2 (untracked content)
#
在这种情况下, 只有部分作出file1.cc中的变化已经上演 /索引下一个承诺。
我运行一个pre-commit脚本做运行风格检查:
#!/bin/bash
git stash -q --keep-index
# Do the checks
RESULT=0
while read status file
do
if python ~/python/cpplint.py "$file"; then
let RESULT=1
fi
done < <(git diff --cached --name-status --diff-filter=ACM | grep -P '\.((cc)|(h)|(cpp)|(c))$' )
git stash pop -q
[ $RESULT -ne 0 ] && exit 1
exit 0
如建议在这里 ,我不分级的藏匿文件之前,我跑我的风格检查,并随后弹出他们。 然而,在一些只在一个文件中的变化上演的情况下,这会导致合并冲突,当我跳出藏匿在pre-commit钩子的结束。
什么是更好的方式来做到这一点? 我想运行对即将提交的文件的版本上演风格检查。
我会避免使用git stash
钩自动。 我发现这是可以使用git show ':filename'
让藏文件的内容。
相反,我用另一个方法:
git diff --cached --name-only --diff-filter=ACMR | while read filename; do
git show ":$filename" | GIT_VERIFY_FILENAME="$filename" verify_copyright \
|| exit $?
done \
|| exit $?
更换git stash -q --keep-index
有:
git diff --full-index --binary > /tmp/stash.$$
git stash -q --keep-index
...和git stash pop -q
有:
git apply --whitespace=nowarn < /tmp/stash.$$` && git stash drop -q
rm /tmp/stash.$$
这将差异保存到临时文件,并使用重新涂抹git apply
在最后。 这个变更仍然冗余保存到藏匿(这是后来放弃了),所以如果出现错误的重新应用,则可以使用检查它们git stash show -p
,而不必去寻找临时文件。
如何门没有积攒承诺
我们使用此.git/hooks/pre-commit
用于检查原子语法包
键位
git checkout-index -a --prefix={{temp_dir}}
这可能/可能不会慢得多/采取更多的空间比藏起来,但自动化与指数食堂似乎天生脆。 也许需要有一个git的contrib脚本这使得软/硬链接树,最小空间为只读,临时索引结账,有利于更好/更快.git/hooks/pre-commit
(或者说, .git/hooks/pre-commit-index
)脚本,使得不需要工作目录的一个完整的第二个副本,就在工作-DIR->指数的变化。
#!/usr/bin/env ruby
require 'tmpdir'
autoload :FileUtils, 'fileutils'
autoload :Open3, 'open3'
autoload :Shellwords, 'shellwords'
# ---- setup
INTERACTIVE = $stdout.tty? || $stderr.tty?
DOT = -'.'
BLOCK_SIZE = 4096
TEMP_INDEX_DIR = Dir.mktmpdir
TEMP_INDEX_DIR_REAL = File.realpath(TEMP_INDEX_DIR)
def cleanup
FileUtils.remove_entry(TEMP_INDEX_DIR) if File.exist? TEMP_INDEX_DIR
end
at_exit { cleanup }
%w[INT TERM PIPE HUP QUIT].each do |sig|
Signal.trap(sig) { cleanup }
end
# ---- functions
def fix_up_dir_output(data)
data.gsub! TEMP_INDEX_DIR_REAL, DOT
data.gsub! TEMP_INDEX_DIR, DOT
data
end
def sh(*args)
Open3.popen3(*args) do |_, stdout, stderr, w_thr|
files = [stdout, stderr]
until files.empty? do
if ready = IO.select(files)
ready[0].each do |f|
begin
data = f.read_nonblock BLOCK_SIZE
data = fix_up_dir_output data
if f.fileno == stderr.fileno
$stderr.write data
else
$stdout.write data
end
rescue EOFError
files.delete f
end
end
end
end
if !(done = w_thr.value).success?
exit(done.exitstatus)
end
end
end
def flags(args)
skip = false
r = []
args.each do |a|
if a[0] == '-' && !skip
a.slice! 0
if a[0] == '-'
skip ||= a[1].nil?
a.slice! 0
r << a unless a.empty?
else # -[^-]+
r += a.split ''
end
end
end
r
end
def less_lint
args = %w[lessc --lint]
args << '--no-color' unless INTERACTIVE
args << 'index.less'
sh(*args)
end
def ensure_git_commit_signed
pcmd = `ps -wwp#{Process.ppid} -ocommand=`.chop
args = flags(Shellwords.split(pcmd)[1..-1])
return unless (args & %w[gpg-sign S]).empty?
$stderr.puts 'All git commits must be GPG-signed'
$stderr.puts " command: #{pcmd}"
exit 1
end
# ---- main
# 1. make sure all commits are signed
ensure_git_commit_signed
# 2. check files that are in the index
sh 'git', 'checkout-index', '-a', "--prefix=#{TEMP_INDEX_DIR}/"
Dir.chdir TEMP_INDEX_DIR do
# 3. make sure all commits contain only legal .less files
less_lint
end