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

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1050 bytes --]

On Mon, Jan 21, 2019 at 12:05:26PM +0000, Pavel Moseev wrote:
> Update of /people/mars/packages/make-initrd.git
> 
> Changes statistics since `2.2.6-alt1' follows:
>  .gear/make-initrd.spec          | 5 ++++-
>  Makefile                        | 5 +++--
>  add.new.module.d/add-new-sha256 | 7 +++++++
>  mk/config.mk.in                 | 1 +
>  tools/add-module                | 6 +++++-
>  tools/sh-functions              | 1 +
>  6 files changed, 21 insertions(+), 4 deletions(-)
> 
> Changelog since `2.2.6-alt1' follows:
> commit 083f06a998ae1c1f2bc0ebeb8c7dc50383d588fb
> Author: Pavel Moseev <mars@altlinux>
> Date:   Mon Jan 21 11:05:16 2019 +0300
> 
>     2.2.6-alt2
>     
>     - Add changes in "add-modules" to fix error with sha256 at i586 architecture

Когда вы пишете %changelog, вы пишете не commit message, а текст,
который увидят люди, которые не увидят весь commit целиком.

Поэтому старайтесь, пожалуйста, писать %changelog так,
чтобы его можно было понять, не заглядывая в код.

Спасибо.


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

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

[Michael Shigorin]

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

> Спасибо.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [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

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

[Sergey]

В письме от понедельник, 21 января 2019 г. 18:53:01 MSK пользователь Alexey 

[...]
> > add-modules: fix sha256 on i586
s/fix/workaround for/
 
> Если мы говорим о форме, то да. Но это сообщение не отражает того, что
> делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
> что не отражено в сообщении.
Понадобился временный костыль, пока мантейнеры соотв. пакетов делают 
правильное решение.

--
Regards, Sergey

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

[Sergey V Turchin]

В письме от понедельник, 21 января 2019 г. 18:53:01 MSK пользователь Alexey 

[...]
> > add-modules: fix sha256 on i586
s/fix/workaround for/
 
> Если мы говорим о форме, то да. Но это сообщение не отражает того, что
> делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
> что не отражено в сообщении.
Понадобился временный костыль, пока мантейнеры соотв. пакетов делают 
правильное решение.

--
Regards, Sergey.

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

[Alexey Gladkov]

On Tue, Jan 22, 2019 at 11:14:41PM +0300, Sergey V Turchin wrote:
> В письме от понедельник, 21 января 2019 г. 18:53:01 MSK пользователь Alexey 
> 
> [...]
> > > add-modules: fix sha256 on i586
> s/fix/workaround for/
>  
> > Если мы говорим о форме, то да. Но это сообщение не отражает того, что
> > делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
> > что не отражено в сообщении.
> Понадобился временный костыль, пока мантейнеры соотв. пакетов делают 
> правильное решение.

Игнорирование ошибки не является даже временным костелём. Это просто
перенесёт проблему на стадию загрузки. Удачи, тем кто таким initrd
воспользуется.

-- 
Rgrds, legion

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

[Alexey Gladkov]

On Tue, Jan 22, 2019 at 11:14:41PM +0300, Sergey V Turchin wrote:
> > Если мы говорим о форме, то да. Но это сообщение не отражает того, что
> > делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
> > что не отражено в сообщении.
> Понадобился временный костыль, пока мантейнеры соотв. пакетов делают 
> правильное решение.

make-initrd c поддержкой kernel.builtin.modinfo уже попал в сизиф. Когда
соберутся ядра с патчиком, то можно будет тестировать решение полностью.

-- 
Rgrds, legion

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

[Anton Farygin]

23.01.2019 0:55, Alexey Gladkov пишет:
> On Tue, Jan 22, 2019 at 11:14:41PM +0300, Sergey V Turchin wrote:
>>> Если мы говорим о форме, то да. Но это сообщение не отражает того, что
>>> делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
>>> что не отражено в сообщении.
>> Понадобился временный костыль, пока мантейнеры соотв. пакетов делают
>> правильное решение.
> make-initrd c поддержкой kernel.builtin.modinfo уже попал в сизиф. Когда
> соберутся ядра с патчиком, то можно будет тестировать решение полностью.
>
Да, это отлично! Может быть и костыль не понадобится.

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

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 663 bytes --]

On Tue, Jan 22, 2019 at 10:55:38PM +0100, Alexey Gladkov wrote:
> On Tue, Jan 22, 2019 at 11:14:41PM +0300, Sergey V Turchin wrote:
> > > Если мы говорим о форме, то да. Но это сообщение не отражает того, что
> > > делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
> > > что не отражено в сообщении.
> > Понадобился временный костыль, пока мантейнеры соотв. пакетов делают 
> > правильное решение.
> 
> make-initrd c поддержкой kernel.builtin.modinfo уже попал в сизиф. Когда
> соберутся ядра с патчиком, то можно будет тестировать решение полностью.

Ещё бы modprobe(8) и modinfo(8) научились с этим работать...


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

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

[Leonid Krivoshein]

23.01.2019 00:55, Alexey Gladkov пишет:
> On Tue, Jan 22, 2019 at 11:14:41PM +0300, Sergey V Turchin wrote:
>>> Если мы говорим о форме, то да. Но это сообщение не отражает того, что
>>> делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
>>> что не отражено в сообщении.
>> Понадобился временный костыль, пока мантейнеры соотв. пакетов делают
>> правильное решение.
> make-initrd c поддержкой kernel.builtin.modinfo уже попал в сизиф. Когда
> соберутся ядра с патчиком, то можно будет тестировать решение полностью.

Вот эти сотни файлов /*.mod и /*.module плюс ещё полтора десятка файлов 
в корне initrd -- это результат чего: костылей или моих неудачных 
экспериментов? :) Вряд ли новой фичи, поскольку я использую зеркало с 
более старой версией make-initrd.


-- 
Best regards,
Leonid Krivoshein.

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

[Alexey Gladkov]

On Wed, Jan 23, 2019 at 04:06:46AM +0300, Leonid Krivoshein wrote:
> 
> 23.01.2019 00:55, Alexey Gladkov пишет:
> > On Tue, Jan 22, 2019 at 11:14:41PM +0300, Sergey V Turchin wrote:
> >>> Если мы говорим о форме, то да. Но это сообщение не отражает того, что
> >>> делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
> >>> что не отражено в сообщении.
> >> Понадобился временный костыль, пока мантейнеры соотв. пакетов делают
> >> правильное решение.
> > make-initrd c поддержкой kernel.builtin.modinfo уже попал в сизиф. Когда
> > соберутся ядра с патчиком, то можно будет тестировать решение полностью.
> 
> Вот эти сотни файлов /*.mod и /*.module плюс ещё полтора десятка файлов 
> в корне initrd -- это результат чего: костылей или моих неудачных 
> экспериментов? :) Вряд ли новой фичи, поскольку я использую зеркало с 
> более старой версией make-initrd.

Это что-то ваше ))

-- 
Rgrds, legion

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

[Alexey Gladkov]

[-- Attachment #1: Type: text/plain, Size: 1210 bytes --]

On Wed, Jan 23, 2019 at 01:33:22AM +0300, Dmitry V. Levin wrote:
> On Tue, Jan 22, 2019 at 10:55:38PM +0100, Alexey Gladkov wrote:
> > On Tue, Jan 22, 2019 at 11:14:41PM +0300, Sergey V Turchin wrote:
> > > > Если мы говорим о форме, то да. Но это сообщение не отражает того, что
> > > > делается в коде. Никакого фикса нет. Есть простое игнорирование проблемы,
> > > > что не отражено в сообщении.
> > > Понадобился временный костыль, пока мантейнеры соотв. пакетов делают 
> > > правильное решение.
> > 
> > make-initrd c поддержкой kernel.builtin.modinfo уже попал в сизиф. Когда
> > соберутся ядра с патчиком, то можно будет тестировать решение полностью.
> 
> Ещё бы modprobe(8) и modinfo(8) научились с этим работать...

Это есть в планах. Нужно сначала патчик на ядро заапстримить.

-- 
Rgrds, legion


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 163 bytes --]