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
next prev 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