Git pre-commit hook : modification/ajout de fichiers

Je suis en train d'écrire un pre-commit hook. Je veux courir php -l à l'encontre de tous les fichiers .extension php. Cependant, je suis coincé.

J'ai besoin d'obtenir une liste de nouveaux/modifiés les fichiers qui sont mis en scène. fichiers supprimés doivent être exclus.

J'ai essayé d'utiliser git diff et git ls-files, mais je pense que j'ai besoin d'un coup de main ici.

InformationsquelleAutor igorw | 2010-03-09
  1. 48

    git diff --cached --name-status affiche un résumé de ce qui est mis en scène, de sorte que vous pouvez facilement exclure les fichiers supprimés, par exemple:

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

    Cela indique que wt-statut.c a été modifié et wt-statut.h a été enlevé dans la zone de transit (index). Donc, pour vérifier que les fichiers n'étaient pas supprimés:

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

    Vous devez sauter à travers les cerceaux de traiter avec les noms de fichiers avec des espaces bien (option-z pour git diff et un peu plus intéressante analyse syntaxique)

    • Merci, c'est un bon début. Cependant, si je modifie un fichier sans mise en scène, c'est toujours affichée. Je suis en cours d'exécution de version git 1.7.0.1.147.g6d84b (récente version personnalisée). Vous ne savez pas si c'est un comportement intentionnel.
    • Que des sons étranges. Le "--en cache" commutateur doit faire afficher uniquement les fichiers qui ont été mis en scène: même si je suis en essais ce avec 1.6.5, il semble surprenant que cela aurait changé... ne "git diff --cached" sur son propre spectacle, le unstaged changements?
    • Après un peu de débogage, j'ai été en mesure de retracer une autre cause. Merci beaucoup!
    • L'exemple a une faute de frappe. $1 != "R" devrait lire $1 != "D"
    • Je serais intéressé mais le lien est mort.
    • Juste à noter que si la seule chose que voulait est le nom du fichier, il y existe-nom-seulement au lieu de --nom de l'état. Pourrait enlever les awk cerceau.
    • Le print $2 partie de la commande awk ne fonctionne que si le nom de fichier ne contient pas d'espaces. Une solution possible pour ce: ... | awk '$1 != "D" { $1=""; sub(FS,""); print $0 }'. Une autre remarque: pour fichier renomme, la git diff commande imprime des ANCIENS et NOUVEAUX noms. Si seulement le nouveau nom est désiré, ajouter un --no-renames option pour git diff, cela permettra de traiter le renommer comme la suppression de l'ancien fichier et l'ajout d'un nouveau fichier.

    InformationsquelleAutor araqnid
  2. 88

    Un peu plus propre façon d'obtenir la même liste est: 

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

    Ce sera le retour de la liste des fichiers qui doivent être vérifiées.

    Mais juste en cours d'exécution php -l sur votre copie de travail ne peut pas être la bonne chose à faire. Si vous faites une partielle s'engager c'est à dire juste en sélectionnant un sous-ensemble des différences entre votre travail actuel et la TÊTE pour la validation, le test sera exécuté sur votre jeu de travail, mais sera certification d'un commit qui n'a jamais existé sur votre disque.

    De le faire correctement, vous devez extraire l'ensemble de la mise en scène de l'image à une température de zone et effectuer le 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

    Voir La construction d'une meilleure pre-commit hook Git pour une autre mise en œuvre.

    • Effectivement, il est possible de canaliser le contenu du fichier à php -l. Et c'est ce que nous nous sommes retrouvés avec. Voir ici: github.com/phpbb/phpbb3/blob/develop-olympus/git-tools/hooks/...
    • De vérifier la syntaxe d'un fichier intermédiaire, vous pouvez utiliser git show :FILENAME | php -l.
    • --diff-filtre doit probablement être "ACMR", parce que les fichiers renommés (R) peut avoir trop de changements.
    InformationsquelleAutor LarryH
  3. 10

    Voici ce que j'utilise pour mon Perl vérifie:

    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

    pour PHP, il ressemblera à ceci:

    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
    • Très bon, mais ne fonctionne pas pour partiellement mise en scène de fichiers, car il lit le fichier en entier.
    InformationsquelleAutor Marian HackMan Marinov
  4. 8

    Aucune des réponses ici de soutenir les noms de fichiers avec des espaces. Le meilleur moyen pour cela est d'ajouter la -z drapeau en combinaison avec xargs -0

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

    C'est ce qui est donné par git dans les échantillons intégrés (voir .git/hooks/pre-commit.exemple de)

    InformationsquelleAutor eddygeek
  5. 0

    git diff --cached n'est pas suffisante si le commit appel a été spécifié avec l'option-a, et il n'y a aucun moyen de déterminer si le drapeau a été lancée dans le crochet. Il serait utile si les arguments de commettre devrait être disponible pour le crochet pour examen.

    • git diff --cached NE semble pas être suffisant. Cependant, je crois que si vous lancez la commande git status --en porcelaine à l'intérieur de votre crochet, tous les fichiers qui seront traitées ne seront pas avoir un blanc ou un ? dans la première position de la sortie. Je n'ai pas entièrement testé, mais jusqu'à présent, il a lieu dans toutes les conditions que j'ai dans mon repo, un mélange de nouvelles, d'ajouts, de modifications de fichiers, où j'essaie de commettre explicite des fichiers, l'ensemble par défaut des fichiers, -un pour tout. Alors pourquoi utiliser git status au lieu de git diff? Je pense que c'est plus facile à analyser.
    • git status --porcelain | grep -E -v '^[? ]'
    • git status --porcelain | perl -ane 'print $F[1],qq(\n) if m/^[ACM] /' est une meilleure réponse. Il a l'avantage d'utiliser un --porcelaine option, garantie de ne jamais changer. Utilisez votre propre analyseur si perl est trop lourd pour vous.
    InformationsquelleAutor mpersico