ALT Linux Distributions development
 help / color / mirror / Atom feed
From: Michael Shigorin <mike@altlinux.org>
To: devel-distro@lists.altlinux.org
Subject: Re: [devel-distro] Пропажа ядерного модуля в профиле из образа
Date: Thu, 5 Dec 2019 12:38:40 +0300
Message-ID: <20191205093840.GD29047@imap.altlinux.org> (raw)
In-Reply-To: <ecb0b09a-072a-97b1-66da-3996fc87eaa3@ya.ru>

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


  parent reply	other threads:[~2019-12-05  9:38 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-05  8:56 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 [this message]
2019-12-05 10:34     ` Sergey V Turchin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191205093840.GD29047@imap.altlinux.org \
    --to=mike@altlinux.org \
    --cc=devel-distro@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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