From: "Антон Мидюков" <midyukov-anton@ya.ru>
To: devel-distro@lists.altlinux.org
Subject: Re: [devel-distro] mkimage-profiles 1.4.9-alt1
Date: Tue, 13 Apr 2021 10:22:37 +0700
Message-ID: <ecec045b-4bcd-e985-8edb-ecf278dbda21@ya.ru> (raw)
In-Reply-To: <b850a2e0-257a-432d-096d-86a3e355aab7@gmail.com>
13.04.2021 02:02, Leonid Krivoshein пишет:
>
>
> 12.04.2021 21:45, Антон Мидюков пишет:
>> 13.04.2021 01:36, Leonid Krivoshein пишет:
>>> 12.04.2021 20:14, Антон Мидюков пишет:
>>>> 12.04.2021 23:52, Leonid Krivoshein пишет:
>>>>> 05.04.2021 20:17, Антон Мидюков пишет:
>>>>>> [...]
>>>>>> - Добавлена поддержка сборки образа img с grub-efi для riscv64 (спасибо arei@);
>>>>>> - Добавлена фича uuid-iso, которая генерирует заданный uuid для iso и передаёт его mkimage (спасибо jqt4@).
>>>>>> Требуется mkimage 0.2.41. Действие фичи ограничено, пока, архитектурами i586, x86_64, aarch64.
>>>>>> Это позволило использовать на этих архитектурах метод загрузки disk с указанием uuid файловой системы;
>>>>> И как при этом изменении будут извлекаться CD-ROM'ы в нужный момент инсталляции? Ведь сейчас эта фича проверяет экспортированный в stage2 METHOD=cdrom.
>>>> А в какой момент они извлекаются? Я пока сталкивался только с извлечением cdrom после завершения установки, перед самой перезагрузкой.
>>>> Я так понимаю, что это нужно для установки с нескольких дисков. Но мы же такие не делаем?
>>> По умолчанию таких мест -- неизведанное множество. :-)
>>> Но на одно такое место мы уже напарывались, выкидывая пропагатор:
>>>
>>> installer/preinstall.d/99-cdrom.sh
>>>
>> Никаких проблем в этом скрипте нет при использовании метода disk.
>
> Это из-за строчки ниже и из-за того, что видимо на этой архитектуре система ставится не с CD-ROM. На самом деле следующий за этой строкой кусок проблемный: он копирует stage2 altinst в свободное место на диске без учёта lowmem, когда altinst итак уже безопасно отделён в /dev/ram3. И где-то в инсталляторе должна быть ответная часть, молчаливо удаляющая этот ненужный сквош с диска.
Этот скрипт выполняется, когда method=cdrom. Соответственно, нужно понять, что плохого при установке с cd-rom может произойти от того, что он не выполняется. Я проверяю в virtualbox, и никаких проблем не наблюдаю. Вот только cd-rom не извлекается.
Параметр lowmem при загрузке altinst не назначается.
Ответная часть в postinstall:
# Eject cdrom
if [ -b "$CDROMDEV" ]; then
sed -i "/ $(printf %s "$destdir" |sed 's,/,\\/,g')/ d" /etc/mtab >/dev/null 2>&1
umount -fl "$image_dir" 2>/dev/null ||:
cmdline_has_arg noeject || action 'Ejecting removable media' eject -p "$CDROMDEV" ||:
fi
она не выполняется при использовании method=disk. Пока вижу лишь одно плохое последствие, cdrom не извлекается в конце установки.
С другой стороны, что хорошего было от того, что такой код выполнялся при установке с флешки?
>
>>> ...
>>> [ "$METHOD" = cdrom ] || exit 0
>>> ...
>>>
>>> Вообще я хотел текстом выше сказать, что что UUID можно (и нужно) привязывать к методу cdrom, а не disk. Ну, просто по логике.
>> Да не нужно. cdrom вымирают. Мы можем сразу в img в mkimage паковать (в будущем).
>
> Как я понимаю, ранее дистрибутивы ALT поставлялись только в виде ISO-образов и даже при сетевой установке это было преимущественным методом. Сейчас мы конечно можем отличать диски по флагу съёмного устройства, но придётся вносить изменения, как минимум, в скрипт remount, где есть спотыкания, когда ставим не из ISO. Да и вообще придётся учитывать разные способы установки и развёртывания, как в первой фазе установщика, так и во второй.
>
Нужно выявлять эти нюансы.
--
С уважением, Антон Мидюков <antohami@altlinux.org>
next prev parent reply other threads:[~2021-04-13 3:22 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-05 17:17 Антон Мидюков
2021-04-06 12:55 ` Michael Shigorin
2021-04-06 13:23 ` Антон Мидюков
2021-04-12 16:52 ` Leonid Krivoshein
2021-04-12 17:14 ` Антон Мидюков
2021-04-12 18:36 ` Leonid Krivoshein
2021-04-12 18:45 ` Антон Мидюков
2021-04-12 19:02 ` Leonid Krivoshein
2021-04-13 3:22 ` Антон Мидюков [this message]
2021-04-13 7:28 ` [devel-distro] про eject (was: mkimage-profiles 1.4.9-alt1) Michael Shigorin
2021-04-13 9:17 ` Антон Мидюков
2021-04-13 10:15 ` [devel-distro] mkimage-profiles 1.4.9-alt1 Anton V. Boyarshinov
2021-04-13 19:13 ` Антон Мидюков
2021-04-13 22:19 ` Leonid Krivoshein
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=ecec045b-4bcd-e985-8edb-ecf278dbda21@ya.ru \
--to=midyukov-anton@ya.ru \
--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