ALT Linux Distributions development
 help / color / mirror / Atom feed
* [devel-distro] Пропажа ядерного модуля в профиле из образа.
@ 2019-12-05  8:56 Sergey V Turchin
  2019-12-05  9:18 ` Антон Мидюков
  0 siblings, 1 reply; 8+ messages in thread
From: Sergey V Turchin @ 2019-12-05  8:56 UTC (permalink / raw)
  To: devel-distro

Привет всем!

У меня в http://git.altlinux.org/people/zerg/packages/?p=mkimage-profiles-kworkstation.git;a=blob;f=features.in/wireless/config.mk указан rtl8723de, но 
в образ дистрибутива молча не попадает. Как такое может быть?

P.S.
Собираю с ядром un-def и в данный момент kernel-modules-rtl8723de-un-def 
отсутствует в репозитории.

-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [devel-distro] Пропажа ядерного модуля в профиле из образа.
  2019-12-05  8:56 [devel-distro] Пропажа ядерного модуля в профиле из образа Sergey V Turchin
@ 2019-12-05  9:18 ` Антон Мидюков
  2019-12-05  9:32   ` Sergey V Turchin
  2019-12-05  9:38   ` Michael Shigorin
  0 siblings, 2 replies; 8+ messages in thread
From: Антон Мидюков @ 2019-12-05  9:18 UTC (permalink / raw)
  To: devel-distro

05.12.2019 15:56, Sergey V Turchin пишет:
> Привет всем!
>
> У меня в http://git.altlinux.org/people/zerg/packages/?p=mkimage-profiles-kworkstation.git;a=blob;f=features.in/wireless/config.mk указан rtl8723de, но
> в образ дистрибутива молча не попадает. Как такое может быть?
>
> P.S.
> Собираю с ядром un-def и в данный момент kernel-modules-rtl8723de-un-def
> отсутствует в репозитории.
>
Потому что отсутствует в репозитории. Действует принцип: есть модуль 
добавляем, нет - ну и фиг с ним.

Согласен, надо хоть warning какой сделать.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [devel-distro] Пропажа ядерного модуля в профиле из образа.
  2019-12-05  9:18 ` Антон Мидюков
@ 2019-12-05  9:32   ` Sergey V Turchin
  2019-12-05  9:41     ` Антон Мидюков
  2019-12-05  9:46     ` Michael Shigorin
  2019-12-05  9:38   ` Michael Shigorin
  1 sibling, 2 replies; 8+ messages in thread
From: Sergey V Turchin @ 2019-12-05  9:32 UTC (permalink / raw)
  To: Distributions development

On Thursday, 5 December 2019 12:18:39 MSK Антон Мидюков wrote:
> 05.12.2019 15:56, Sergey V Turchin пишет:
> > Привет всем!
> > 
> > У меня в
> > http://git.altlinux.org/people/zerg/packages/?p=mkimage-profiles-kworksta
> > tion.git;a=blob;f=features.in/wireless/config.mk указан rtl8723de, но в
> > образ дистрибутива молча не попадает. Как такое может быть?
> > 
> > P.S.
> > Собираю с ядром un-def и в данный момент kernel-modules-rtl8723de-un-def
> > отсутствует в репозитории.
> 
> Потому что отсутствует в репозитории. 
> Действует принцип: есть модуль добавляем, нет - ну и фиг с ним.
Это как-то слишком. Я об этом узнал от техподдержки. Клёво, да?

> Согласен, надо хоть warning какой сделать.
Обязательно надо сделать возможность прерывать сборку с ошибкой. Баг вешать?

-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [devel-distro] Пропажа ядерного модуля в профиле из образа
  2019-12-05  9:18 ` Антон Мидюков
  2019-12-05  9:32   ` Sergey V Turchin
@ 2019-12-05  9:38   ` Michael Shigorin
  2019-12-05 10:34     ` Sergey V Turchin
  1 sibling, 1 reply; 8+ messages in thread
From: Michael Shigorin @ 2019-12-05  9:38 UTC (permalink / raw)
  To: devel-distro

On Thu, Dec 05, 2019 at 04:18:39PM +0700, Антон Мидюков wrote:
> > У меня в
> > http://git.altlinux.org/people/zerg/packages/?p=mkimage-profiles-kworkstation.git;a=blob;f=features.in/wireless/config.mk
> > указан rtl8723de, но в образ дистрибутива молча не попадает.
> > Как такое может быть?

Из KFLAVOURS и KMODULES в image.in/functions.mk порождается
регулярное выражение -- кто подошёл, тот и попал.

> > P.S.  Собираю с ядром un-def и в данный момент
> > kernel-modules-rtl8723de-un-def отсутствует в репозитории.
> Потому что отсутствует в репозитории.

Собственно, а чего ты хотел -- или вопрос именно в _молча_?

> Действует принцип: есть модуль добавляем, нет - ну и фиг с ним.
> Согласен, надо хоть warning какой сделать.

С текущей реализацией kpackages это не будет тривиально.

Думаю, тут есть примерно два (сочетаемых) пути:

1) сделать тесты по содержимому, см., к примеру,
   features.in/rescue/rescue/image-scripts.d/00-test-rescue;

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

Для совсем хорошего первого варианта хорошо бы сперва
привести в приличный вид подсистему логирования:

а) сейчас все warning'и попадают в build.log, где их мало
   кто будет искать при _каждой_ сборке (или просто никто);

б) сейчас есть только ручка DEBUG о трёх положениях (0, 1, 2),
   а хорошо бы придумать или набор ручек, или тщательно
   выверенный "светофор", когда можно было бы системно
   включать, например, -x для запускаемых скриптов
   (сюда же и проблемка с выхлопом при создании чрутов,
   на которую обратил внимание boyarsh@ и которую я признал,
   но сейчас не возьмусь точно формализовать);

в) возможно, после решения (а) и фильтра заведомого шума
   получится сделать что-то вроде -Werror.

Но если тесты будут выдавать не предупреждения, а обрыв сборки,
то можно и далее откладывать логи на светлое будущее (tm).

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


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [devel-distro] Пропажа ядерного модуля в профиле из образа.
  2019-12-05  9:32   ` Sergey V Turchin
@ 2019-12-05  9:41     ` Антон Мидюков
  2019-12-05  9:50       ` Michael Shigorin
  2019-12-05  9:46     ` Michael Shigorin
  1 sibling, 1 reply; 8+ messages in thread
From: Антон Мидюков @ 2019-12-05  9:41 UTC (permalink / raw)
  To: devel-distro

05.12.2019 16:32, Sergey V Turchin пишет:
> On Thursday, 5 December 2019 12:18:39 MSK Антон Мидюков wrote:
>> 05.12.2019 15:56, Sergey V Turchin пишет:
>>> Привет всем!
>>>
>>> У меня в
>>> http://git.altlinux.org/people/zerg/packages/?p=mkimage-profiles-kworksta
>>> tion.git;a=blob;f=features.in/wireless/config.mk указан rtl8723de, но в
>>> образ дистрибутива молча не попадает. Как такое может быть?
>>>
>>> P.S.
>>> Собираю с ядром un-def и в данный момент kernel-modules-rtl8723de-un-def
>>> отсутствует в репозитории.
>> Потому что отсутствует в репозитории.
>> Действует принцип: есть модуль добавляем, нет - ну и фиг с ним.
> Это как-то слишком. Я об этом узнал от техподдержки. Клёво, да?
>
>> Согласен, надо хоть warning какой сделать.
> Обязательно надо сделать возможность прерывать сборку с ошибкой. Баг вешать?
Да, багу на mkimage-profiles. Так было с первых коммитов 
mkimage-profiles, как я понимаю. Придётся сильно актуализировать 
профили. Использование регэкспа позволяло не заморачиваться с 
актуализацией модулей. Также это создаст сложности с портированием. 
Придётся под каждое ядро подстраиваться.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [devel-distro] Пропажа ядерного модуля в профиле из образа
  2019-12-05  9:32   ` Sergey V Turchin
  2019-12-05  9:41     ` Антон Мидюков
@ 2019-12-05  9:46     ` Michael Shigorin
  1 sibling, 0 replies; 8+ messages in thread
From: Michael Shigorin @ 2019-12-05  9:46 UTC (permalink / raw)
  To: devel-distro

On Thu, Dec 05, 2019 at 12:32:14PM +0300, Sergey V Turchin wrote:
> > Согласен, надо хоть warning какой сделать.
> Обязательно надо сделать возможность прерывать сборку с ошибкой.
> Баг вешать?

Тест сделай, см. соседнее письмо -- может, даже быстрее выйдет :-)

Что-то вроде такого:

--- features.in/kernel/...
#!/bin/sh -efu

# вот здесь надо придумать, как раскрыть тот регэкс
# или получить эквивалентный список другим макаром
kpackages() {
        if ! rpmquery ... >&/dev/null; then
                echo "** some kernel packages missing"
                # а здесь -- вычесть выданное rpm -qa
                echo "....."
                exit 1
        fi >&2
}

case "${GLOBAL_TEST:-}" in
*kpackages*)
        kpackages;;
esac
--- по мотивам features.in/rescue/rescue/image-scripts.d/00-test-rescue

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


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [devel-distro] Пропажа ядерного модуля в профиле из образа.
  2019-12-05  9:41     ` Антон Мидюков
@ 2019-12-05  9:50       ` Michael Shigorin
  0 siblings, 0 replies; 8+ messages in thread
From: Michael Shigorin @ 2019-12-05  9:50 UTC (permalink / raw)
  To: devel-distro

On Thu, Dec 05, 2019 at 04:41:46PM +0700, Антон Мидюков wrote:
> >> Согласен, надо хоть warning какой сделать.
> > Обязательно надо сделать возможность прерывать сборку с ошибкой. Баг вешать?
> Да, багу на mkimage-profiles. Так было с первых коммитов 
> mkimage-profiles, как я понимаю. Придётся сильно
> актуализировать профили. Использование регэкспа позволяло не
> заморачиваться с актуализацией модулей. Также это создаст
> сложности с портированием.  Придётся под каждое ядро
> подстраиваться.

Стоп!  ЛОМАТЬ нельзя, попросили ВОЗМОЖНОСТЬ.

Мы достаточно намучались с прибитыми гвоздями списками
в m-p-d, когда от флуктуаций в модулях из RPMS.addon
сборка исошки обламывалась полчаса спустя.

Поэтому сделано так -- намеренно.  Кому надо -- может
в любой момент точно так же в списках пакетов прибить
гвоздиком.

А вот ВКЛЮЧАЕМЫЕ тесты сделать -- хорошо: довёл до
сборчивости, включил такой "-Werror", выяснил остатки
проблем -- всё, можно отгружать выпуск.

PS: хоть кто-то ещё заметил, что вообще-то это вопрос
на то, с какими ядрами и почему идут дистрибутивы,
а также на то, как и кем задаётся и обеспечивается
набор модулей для них, которые считаются важными?
(2 klark: прямо пересекается с поднятой тобою темой)

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


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [devel-distro] Пропажа ядерного модуля в профиле из образа
  2019-12-05  9:38   ` Michael Shigorin
@ 2019-12-05 10:34     ` Sergey V Turchin
  0 siblings, 0 replies; 8+ messages in thread
From: Sergey V Turchin @ 2019-12-05 10:34 UTC (permalink / raw)
  To: Distributions development

On Thursday, 5 December 2019 12:38:40 MSK Michael Shigorin wrote:
> On Thu, Dec 05, 2019 at 04:18:39PM +0700, Антон Мидюков wrote:
> > > У меня в
> > > http://git.altlinux.org/people/zerg/packages/?p=mkimage-profiles-kworkst
> > > ation.git;a=blob;f=features.in/wireless/config.mk указан rtl8723de, но в
> > > образ дистрибутива молча не попадает.
> > > Как такое может быть?
> 
> Из KFLAVOURS и KMODULES в image.in/functions.mk порождается
> регулярное выражение -- кто подошёл, тот и попал.
> 
> > > P.S.  Собираю с ядром un-def и в данный момент
> > > kernel-modules-rtl8723de-un-def отсутствует в репозитории.
> > 
> > Потому что отсутствует в репозитории.
> 
> Собственно, а чего ты хотел -- или вопрос именно в _молча_?
Я хотел это узнавать до окончания сборки образа, а не потом от техподдержки.

[...]

-- 
Regards, Sergey.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2019-12-05 10:34 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-05  8:56 [devel-distro] Пропажа ядерного модуля в профиле из образа Sergey V Turchin
2019-12-05  9:18 ` Антон Мидюков
2019-12-05  9:32   ` Sergey V Turchin
2019-12-05  9:41     ` Антон Мидюков
2019-12-05  9:50       ` Michael Shigorin
2019-12-05  9:46     ` Michael Shigorin
2019-12-05  9:38   ` Michael Shigorin
2019-12-05 10:34     ` Sergey V Turchin

ALT Linux Distributions development

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-distro/0 devel-distro/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel-distro devel-distro/ http://lore.altlinux.org/devel-distro \
		devel-distro@lists.altlinux.org devel-distro@lists.altlinux.ru devel-distro@lists.altlinux.com
	public-inbox-index devel-distro

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel-distro


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git