Как нам обустроить патчи

[beldmit]

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

Вводная. У меня есть два основных пакета, с которыми я на работе ковыряюсь, OpenSSH и OpenSSL. И там и там 60+ патчей, которые при каждом ребейзе приносят адскую боль. OpenSSL я знаю хорошо и поэтому с этой болью я почти смирился, OpenSSH я знаю средне.

Случаи, когда код переписан содержательно, всегда требуют индивидуального подхода и деваться от этого некуда. Но хочется минимизировать технические вещи.

Очевидно, что чисто по времени при ребейзе дофига жрут сцепленные патчи. Мы поправили строку из контекста, и теперь у нас конфликт. Мы решили конфликт, нарушив тем самым контекст следующего патча. Возможно, эти два патча стоит склеить в один (или нет, если они решают разные проблемы). Но для этого их надо идентифицировать как "сцепленные".

Ещё очевидно, что патчи к одной подсистеме лучше прикладывать подряд, потому что контекст (в голове). И опять же склеить там, где возможно.

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

Номера затронутых строк и строк контекста так просто из patch/diff не получить. Во всяком случае, я не нашёл. Ну теоретически эту часть я могу хоть на Perl написать, не rocket science. Найти сцепленные патчи так можно, и дальше уже решать, что с этим делать.

Следующий интересный вопрос - а что ещё с этим можно сделать, чтобы улучшить поддерживаемость набора патчей in the long run.

На уровне разработки я нашёл совершенно чудный инструмент git absorb, который разбирает свежие изменения на fixup-ы и новые содержательные. Не знаю, как он потянет 60+ коммитов, конечно.

Вот тут пишут про замену diff для GitHub, но в основном для web-интерфейса.

Difftastic - продвинутый diff, не по строкам, а по логическим блокам

Mergiraf - продвинутый merge.

В дискуссию приглашается, например, spamsink

Comments

[yurikhan]

На тему git absorb ещё есть git autofixup. Кажется, с немного другим принципом действия, но той же решаемой задачей.

[vitus_wagner]

Ну как не получить номера строк?

Берем типичный патч, context unified diff. И ччто мы там видим?

--- a/tests/playTests.sh
+++ b/tests/playTests.sh
@@ -1857,8 +1857,13 @@

Вот то что после @@ это номер первой строки блока и через запятую количество затрнутых строк.

[dimas]

А с какой стороны у тебя больше патчей - твои локальные или что приходит из мейнстрима? Может надо просто в другую сторону мержить?

Потом, я бы посмотрел на твой набор патчей, если таки их меньше, чем мейнстрим меняется ... Возможно, их можно поделить на части, которые будут проще автоматом применяться к мейнстриму.

dft и mf я себе в конфигурацию прикрутил, но пока особо не использовал.

[alexott]

Stacked branches не помогут?
https://andrewlock.net/working-with-stacked-branches-in-git-is-easier-with-update-refs/

[juan_gandhi]

Да я вообще пока что не вижу такой науки, которая бы решала эту задачу.

[livelight]

Но зачем ребейз?!