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