Re: [devel] peer review

[Damir Shayhutdinov <damir@altlinux.org>]

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

>> И первоначальный push наша рассылка не показывает,
>
> А вы хотите, чтобы показывала?
Ну хотя бы первоначальный .spec.

>> да и часто изменения в спеке не показываются из-за большого
>> количества изменений в апстримных исходниках.
>
> Вы думаете, что имеет смысл рассылать многомегабайтные письма?
Нет, конечно не имеет. Хотя изменения в файлах .spec и .patch
(появление, удаление, изменение) стоило бы показывать в любом случае.

> Существует ли способ отделить изменения, выполненные мейнтейнером,
> от изменений в апстримных исходниках, если мейнтейнер не сделал
> этого специально?
committer.email == "*@altlinux.*" ?