#39611 – Использовать META_VOL_ID для @distro@

Bug 39611 - Использовать META_VOL_ID для @distro@

Summary: Использовать META_VOL_ID для @distro@

Status:	CLOSED FIXED

Alias:	None

Product:	Sisyphus
Classification:	Development
Component:	mkimage-profiles (show other bugs)
Version:	unstable
Hardware:	x86_64 Linux

Importance:	P5 enhancement
Assignee:	Антон Мидюков
QA Contact:	qa-sisyphus

URL:
Keywords:

Depends on:
Blocks:

Reported:	2021-01-27 11:28 MSK by Sergey V Turchin
Modified:	2021-02-16 08:27 MSK (History)
CC List:	2 users (show)

See Also:

Attachments
патч (343 bytes, patch) 2021-01-27 11:28 MSK, Sergey V Turchin	no flags	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description

Sergey V Turchin 2021-01-27 11:28:23 MSK

Created attachment 9156 [details]
патч

Предлагаю использовать META_VOL_ID, а не META_VOL_SET, т.к. у всех в текущем варианте "ALT".

Comment 1

Антон Мидюков 2021-01-27 17:07:26 MSK

Сделал такой коммит:
http://git.altlinux.org/people/antohami/packages/mkimage-profiles.git?p=mkimage-profiles.git;a=commit;h=8fb0f630eb723cea9a655c94722b836eddb60e9a

Если META_VOL_ID будет не задан (регулярки/стартеркиты), то будет как и раньше $(RELNAME). А если задан, то должно быть так, как надо. Прошу проверить.

Comment 2

Sergey V Turchin 2021-01-28 11:14:40 MSK

Кавычки потерял. Не
menuentry $"Install "@distro@
, а
menuentry $"Install "'@distro@'

Comment 3

Антон Мидюков 2021-01-28 11:16:14 MSK

(Ответ для Sergey V Turchin на комментарий #2)
> Кавычки потерял. Не
> menuentry $"Install "@distro@
> , а
> menuentry $"Install "'@distro@'

Так, а что эти кавычки одинарные в данном случае делают?

Comment 4

Sergey V Turchin 2021-01-28 11:25:50 MSK

> Так, а что эти кавычки одинарные в данном случае делают?
Как везде, показывают, от кель и до кель, а не до 1-го пробела.

Comment 5

Антон Мидюков 2021-01-28 11:43:38 MSK

(Ответ для Sergey V Turchin на комментарий #4)
> > Так, а что эти кавычки одинарные в данном случае делают?
> Как везде, показывают, от кель и до кель, а не до 1-го пробела.

Переделал коммит. Посмотрите последний коммит ветки next.

Comment 6

Sergey V Turchin 2021-01-28 12:10:49 MSK

Остальное всё было ok.

Comment 7

Sergey V Turchin 2021-01-28 12:22:43 MSK

Боюсь, пробел после "@'" может быть лишний в
$"VNC install "'@distro@' $"(edit to set server IP address)"

Comment 8

Антон Мидюков 2021-01-28 13:35:27 MSK

(Ответ для Sergey V Turchin на комментарий #7)
> Боюсь, пробел после "@'" может быть лишний в
> $"VNC install "'@distro@' $"(edit to set server IP address)"

Да, действительно. Исправил последний коммит и проверил. Теперь должно быть норм.

Comment 9

Sergey V Turchin 2021-01-28 13:40:32 MSK

Да, теперь вроде всё.

Comment 10

Repository Robot 2021-02-15 17:24:09 MSK

mkimage-profiles-1.4.7-alt1 -> sisyphus:

 Mon Feb 15 2021 Anton Midyukov <antohami@altlinux> 1.4.7-alt1
 - Add kernel modules for support Raspberry Pi 4 (mainline kernel)
 - grub: Markup configuration files for translation (thanks Ivan Razzhivin)
 - grub: Use META_VOL_ID for @distro@ instead RELNAME if available
   (ALT bug 39611, 39612)
 - tar2fs: Enable secure-boot support for x86_64, add riscv64 support
 - Add initrd features for rootfs: kbd rdshell rootfs
 - desktop+mate: list is fixed,  as removed metapackage mate-default
 - grub: use single boot/grub/grub.cfg in ISO
 - Add support grub-pc for bootloading ISO (experimentall)
 - Removed kernel modules that haven't been built for a long time