[devel] [SCM] packages/make-initrd: heads/sisyphus

[Alexey Gladkov]

On Mon, Jan 21, 2019 at 06:39:26PM +0300, Michael Shigorin wrote:
> On Mon, Jan 21, 2019 at 05:16:48PM +0300, Dmitry V. Levin wrote:
> > >     - Add changes in "add-modules" to fix error with sha256 at i586 architecture
> > Когда вы пишете %changelog, вы пишете не commit message,
> > а текст, который увидят люди, которые не увидят весь commit
> > целиком.
> 
> Напомню ещё один момент про описания коммитов:
> документировать стоит _причину_, _намерение_ или
> ещё что-либо неочевидное из реализации, которая
> и так задокументирована кодом.
> 
> > Поэтому старайтесь, пожалуйста, писать %changelog так,
> > чтобы его можно было понять, не заглядывая в код.
> 
> ...в данном разе, возможно, хватило бы чего-то вроде
> 
> add-modules: fix sha256 on i586

Если мы говорим о форме, то да. Но это сообщение не отражает того, что
делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
что не отражено в сообщении.

-- 
Rgrds, legion