Git pre-commit hook: file modificati/aggiunti

Question

Sto scrivendo un hook pre-commit. Voglio eseguire php -l su tutti i file con estensione .php. Comunque sono bloccato.

Ho bisogno di ottenere un elenco di file nuovi/modificati che sono messi in scena. i file cancellati dovrebbero essere esclusi.

Ho provato a utilizzare git diff e git ls-files, ma penso di aver bisogno di una mano qui.

Answer 1

git diff --cached --name-status mostrerà un riassunto di ciò che è messo in scena, in modo da poter escludere i file rimossi, ad esempio:

M  wt-status.c 
D  wt-status.h 

Questo indica che la WT-status.c è stato modificato e WT-status.h è stato rimosso nel area di staging (indice). Quindi, per controllare solo i file che non sono stati rimossi:

steve@arise:~/src/git <master>$ git diff --cached --name-status | awk '$1 != "D" { print $2 }' 
wt-status.c 
wt-status.h 

Si dovrà fare i salti mortali in più a che fare con nomi di file con spazi in se (opzione -z a git diff e un po 'più interessante di analisi)

Answer 2

Un modo un po 'più ordinato di ottenere la stessa lista è:

git diff --cached --name-only --diff-filter=ACM 

Ciò restituirà l'elenco dei file che devono essere controllati.

Ma l'esecuzione di php -l sulla copia di lavoro potrebbe non essere la cosa giusta da fare. Se stai facendo un commit parziale, ovvero selezionando solo un sottoinsieme delle differenze tra il tuo working set corrente e HEAD per il commit, allora il test verrà eseguito sul tuo working set, ma certifica un commit che non è mai esistito sul tuo disco.

Per eseguire correttamente l'operazione, è necessario estrarre l'intera immagine messa in scena in un'area temporanea ed eseguire il test lì.

rm -rf $TEMPDIR 
mkdir -p $TEMPDIR 
git checkout-index --prefix=$TEMPDIR/ -af 
git diff --cached --name-only --diff-filter=ACM | xargs -n 1 -I '{}' \bin\echo TEMPDIR/'{}' | grep \\.php | xargs -n 1 php -l 

Vedere Building a better pre-commit hook for Git per un'altra implementazione.

Answer 3

Ecco quello che uso per i miei assegni Perl:

git diff --cached --name-status | while read st file; do 
     # skip deleted files 
     if [ "$st" == 'D' ]; then continue; fi 
     # do a check only on the perl files 
     if [[ "$file" =~ "(.pm|.pl)$" ]] && ! perl -c "$file"; then 
       echo "Perl syntax check failed for file: $file" 
       exit 1 
     fi 
done 

per PHP che sarà simile a questa:

git diff --cached --name-status | while read st file; do 
     # skip deleted files 
     if [ "$st" == 'D' ]; then continue; fi 
     # do a check only on the php files 
     if [[ "$file" =~ ".php$" ]] && ! php -l "$file"; then 
       echo "PHP syntax check failed for file: $file" 
       exit 1 
     fi 
done 

Answer 4

git diff --cached non è sufficiente se è stato specificato la chiamata commit con il flag -a, e non c'è modo di determinare se quel flag è stato gettato nel gancio. Sarebbe di grande aiuto se gli argomenti da impegnare fossero disponibili per l'esame.

Answer 5

Nessuna delle risposte qui supporta i nomi file con spazi. Il modo migliore per che deve aggiungere il flag -z in combinazione con xargs -0

git diff --cached --name-only --diff-filter=ACM -z | xargs -0 ... 

Questo è ciò che è dato da git in incorporato campioni (vedi .git/ganci/pre-commit.sample)