Окончания строк перепутаны в Git - как отслеживать изменения из другой ветви после огромного исправления окончания строки? - PullRequest
24 голосов
/ 18 июня 2009

Мы работаем с сторонним движком PHP, который получает регулярные обновления. Релизы хранятся в отдельной ветке в git, а наш ветвь - главная ветвь.

Таким образом, мы сможем применить патчи к нашему форку из новых версий движка.

Моя проблема в том, что после многих коммитов в нашу ветку я понял, что первоначальный импорт движка был сделан с окончаниями строк CRLF.

Я конвертировал каждый файл в LF, но это сделало огромный коммит с удалением 100 тыс. Строк и добавлением 100 тыс. Строк, что явно нарушает то, что мы собирались сделать: легко объединять патчи из заводских выпусков этого стороннего движка.

Что я должен знать? Как я могу это исправить? У меня уже есть сотни коммитов на нашей вилке.

Что было бы хорошо, так это как-то сделать фиксацию конца строки после первоначального импорта и перед ветвлением нашего собственного форка, и удалить этот огромный коммит окончания строки позже в истории.

Однако я не знаю, как это сделать в Git.

Спасибо!

Ответы [ 5 ]

34 голосов
/ 30 июня 2009

Мне наконец-то удалось это решить.

Ответ:

git filter-branch --tree-filter '~/Scripts/fix-line-endings.sh' -- --all

fix-line-endings.sh содержит:

#!/bin/sh
find . -type f -a \( -name '*.tpl' -o -name '*.php' -o -name '*.js' -o -name '*.css' -o -name '*.sh' -o -name '*.txt' -iname '*.html' \) | xargs fromdos

После того как все окончания строк были зафиксированы во всех деревьях во всех коммитах, я сделал интерактивную перебазировку и удалил все коммиты, которые фиксировали окончания строк.

Теперь мой репо чистый и свежий, готовый к работе :)

Примечание для посетителей: не делайте этого, если ваш репо был перемещен / клонирован, потому что это плохо испортит!

4 голосов
/ 08 января 2010

В дальнейшем избегайте этой проблемы с настройкой core.autocrlf, задокументированной в git config --help:

core.autocrlf

Если true, git преобразует CRLF в конце строк в текстовых файлах в LF при чтении из файловой системы и обратном преобразовании при записи в файловую систему. Переменная может быть установлена ​​на input, в этом случае преобразование происходит только при чтении из файловой системы, но файлы записываются с LF в конце строк. Файл считается «текстовым» (, т.е. должен подвергаться механизму autocrlf) на основе атрибута файла crlf, или, если crlf не указан, на основе содержимого файла. Смотрите gitattributes .

3 голосов
/ 18 июня 2009

Вы смотрели на git rebase?

Вам потребуется заново создать историю своего хранилища следующим образом:

  • фиксирует исправления ограничителя строки
  • начать перебазирование
  • оставить сторонний коммит импорта первым
  • применить исправления ограничителя строки
  • применить другие патчи

Что вам нужно понять, так это то, что это сломает всех нижестоящих репозиториев - клонированных из вашего родительского репо. В идеале вы начнете с нуля с этими.


Обновление : пример использования:

target=`git rev-list --max-count=3 HEAD | tail -n1`
get rebase -i $target

Запустит сессию перебазирования за последние 3 коммита.

2 голосов
/ 09 января 2010

мы избежим этой проблемы в будущем:

1) каждый использует редактор, который удаляет конечные пробелы, и мы сохраняем все файлы с помощью LF.

2) если 1) происходит сбой (может - кто-то случайно сохранит его в CRLF по любой причине), у нас есть скрипт предварительной фиксации, который проверяет символы CRLF:

#!/bin/sh
#
# An example hook script to verify what is about to be committed.
# Called by git-commit with no arguments.  The hook should
# exit with non-zero status after issuing an appropriate message if
# it wants to stop the commit.
#
# To enable this hook, rename this file to "pre-commit" and set executable bit

# original by Junio C Hamano

# modified by Barnabas Debreceni to disallow CR characters in commits


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

crlf=0

IFS="
"
for FILE in `git diff-index --cached $against`
do
    fhash=`echo $FILE | cut -d' ' -f4`
    fname=`echo $FILE | cut -f2`

    if git show $fhash | grep -EUIlq $'\r$'
    then
        echo $fname contains CRLF characters
        crlf=1
    fi
done

if [ $crlf -eq 1 ]
then
    echo Some files have CRLF line endings. Please fix it to be LF and try committing again.
    exit 1
fi

exec git diff-index --check --cached $against --

Этот скрипт использует GNU grep и работает в Mac OS X, однако его следует протестировать перед использованием на других платформах (у нас были проблемы с Cygwin и BSD grep)

3) Если мы обнаружим какие-либо пробельные ошибки, мы используем следующий скрипт для ошибочных файлов:

#!/usr/bin/env php
<?php

    // Remove various whitespace errors and convert to LF from CRLF line endings
    // written by Barnabas Debreceni
    // licensed under the terms of WFTPL (http://en.wikipedia.org/wiki/WTFPL)

    // handle no args
    if( $argc <2 ) die( "nothing to do" );


    // blacklist

    $bl = array( 'smarty' . DIRECTORY_SEPARATOR . 'templates_c' . DIRECTORY_SEPARATOR . '.*' );

    // whitelist

    $wl = array(    '\.tpl', '\.php', '\.inc', '\.js', '\.css', '\.sh', '\.html', '\.txt', '\.htc', '\.afm',
                    '\.cfm', '\.cfc', '\.asp', '\.aspx', '\.ascx' ,'\.lasso', '\.py', '\.afp', '\.xml',
                    '\.htm', '\.sql', '\.as', '\.mxml', '\.ini', '\.yaml', '\.yml'  );

    // remove $argv[0]
    array_shift( $argv );

    // make file list
    $files = getFileList( $argv );

    // sort files
    sort( $files );

    // filter them for blacklist and whitelist entries

    $filtered = preg_grep( '#(' . implode( '|', $wl ) . ')$#', $files );
    $filtered = preg_grep( '#(' . implode( '|', $bl ) . ')$#', $filtered, PREG_GREP_INVERT );

    // fix whitespace errors
    fix_whitespace_errors( $filtered );





    ///////////////////////////////////////////////////////////////////////////////////////////////
    ///////////////////////////////////////////////////////////////////////////////////////////////


    // whitespace error fixer
    function fix_whitespace_errors( $files ) {
        foreach( $files as $file ) {

            // read in file
            $rawlines = file_get_contents( $file );

            // remove \r
            $lines = preg_replace( "/(\r\n)|(\n\r)/m", "\n", $rawlines );
            $lines = preg_replace( "/\r/m", "\n", $lines );

            // remove spaces from before tabs
            $lines = preg_replace( "/\040+\t/m", "\t", $lines );

            // remove spaces from line endings
            $lines = preg_replace( "/[\040\t]+$/m", "", $lines );

            // remove tabs from line endings
            $lines = preg_replace( "/\t+$/m", "", $lines );

            // remove EOF newlines
            $lines = preg_replace( "/\n+$/", "", $lines );

            // write file if changed and set old permissions
            if( strlen( $lines ) != strlen( $rawlines )){

                $perms = fileperms( $file );

                // Uncomment to save original files

                //rename( $file, $file.".old" );
                file_put_contents( $file, $lines);
                chmod( $file, $perms );
                echo "${file}: FIXED\n";
            } else {
                echo "${file}: unchanged\n";
            }

        }
    }

    // get file list from argument array
    function getFileList( $argv ) {
        $files = array();
        foreach( $argv as $arg ) {
          // is a direcrtory
            if( is_dir( $arg ) )  {
                $files = array_merge( $files, getDirectoryTree( $arg ) );
            }
            // is a file
            if( is_file( $arg ) ) {
                $files[] = $arg;
            }
        }
        return $files;
    }

    // recursively scan directory
    function getDirectoryTree( $outerDir ){
        $outerDir = preg_replace( ':' . DIRECTORY_SEPARATOR . '$:', '', $outerDir );
        $dirs = array_diff( scandir( $outerDir ), array( ".", ".." ) );
        $dir_array = array();
        foreach( $dirs as $d ){
            if( is_dir( $outerDir . DIRECTORY_SEPARATOR . $d ) ) {
                $otherdir = getDirectoryTree( $outerDir . DIRECTORY_SEPARATOR . $d );
                $dir_array = array_merge( $dir_array, $otherdir );
            }
            else $dir_array[] = $outerDir . DIRECTORY_SEPARATOR . $d;
        }
        return $dir_array;
    }
?>
2 голосов
/ 18 июня 2009

Одним из решений (не обязательно лучшим) будет использование git-filter-branch для перезаписи истории, чтобы всегда использовать правильные окончания строк. Это должно быть лучшим решением для интерактивного перебазирования, по крайней мере, для большего количества коммитов; также может быть проще справляться со слияниями с помощью git-filter-branch.

Это, конечно, при условии, что история была не опубликована (хранилище не было клонировано).

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...