Bug 43186

Summary: Groups multiple lists
Product: Sisyphus Reporter: Sergey V Turchin <zerg>
Component: installer-alterator-pkgAssignee: Mikhail Efremov <sem>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: enhancement    
Priority: P5 CC: antohami, cas, jqt4, manowar, mike, sem, shaba, zerg
Version: unstable   
Hardware: all   
OS: Linux   

Description Sergey V Turchin 2022-07-08 11:02:04 MSK
Предлагаю добавить возможность указывать несколько спиской пакетов в dekstop-файлах групп выбора пакетов.
Тогда форимровать списки можно будет более гибко, выделяя отдельные компоненты, которые можно пихать в несколько групп.

Например, вместо
X-Alterator-PackageList=workstation/games
может быть что-то типа
X-Alterator-PackageList=workstation/games;workstation/wine;workstation/game-32bit-libs;
Comment 1 Sergey V Turchin 2022-07-08 11:03:26 MSK
А вместо
X-Alterator-PackageList=workstation/emulators
может быть что-то типа
X-Alterator-PackageList=workstation/emulators;workstation/wine;
Comment 2 Антон Мидюков 2022-07-08 11:05:54 MSK
А installer-alterator-pkg уже умеет такое? Это же ему устанавливать.
Comment 3 Sergey V Turchin 2022-07-08 11:09:24 MSK
> А installer-alterator-pkg уже умеет такое? Это же ему устанавливать.
Я догадываюсь, что это не в одном месте надо делать.
Значит, его тоже допилить надо.
Comment 4 Антон Мидюков 2022-07-08 11:11:42 MSK
(Ответ для Sergey V Turchin на комментарий #3)
> > А installer-alterator-pkg уже умеет такое? Это же ему устанавливать.
> Я догадываюсь, что это не в одном месте надо делать.
> Значит, его тоже допилить надо.

Его в первую очередь.
Comment 5 Антон Мидюков 2022-07-12 08:49:46 MSK
Сделал примерную реализацию в installer-alterator-pkg:
[#303510] TESTED (try 3) installer-alterator-pkg.git=3.0.1-alt1

Теперь можно вот так:
X-Alterator-PackageList=workstation/emulators;workstation/wine

но не так:
X-Alterator-PackageList=workstation/emulators;workstation/wine;

Для m-p коммит нужен такой:
https://git.altlinux.org/people/antohami/packages/mkimage-profiles.git?p=mkimage-profiles.git;a=commit;h=61b1832a4feeb3799126e45e4ff9ef2fa521bc9d
Comment 6 Sergey V Turchin 2022-07-12 09:39:39 MSK
(Ответ для Антон Мидюков на комментарий #5)
> Теперь можно вот так:
> X-Alterator-PackageList=workstation/emulators;workstation/wine
Круто! Сечас буду пробовать.
 
> но не так:
> X-Alterator-PackageList=workstation/emulators;workstation/wine;
Ок, но так правильнее. В XDG, например, обязательный символ в конце.
Comment 7 Sergey V Turchin 2022-07-12 11:42:19 MSK
> Теперь можно вот так:
> X-Alterator-PackageList=workstation/emulators;workstation/wine
Проверил -- работает такое:
https://git.altlinux.org/people/zerg/packages/?p=mkimage-profiles-kworkstation.git;a=commitdiff;h=2c97247caf37d74bea0f8d70bcaae2a6c246f40c
Comment 9 Sergey V Turchin 2022-07-12 12:10:22 MSK
А последним символом ";" будем что-то делать? Я, в принципе, не против и без него, т.к. сама возможность уже есть и работает.
Comment 10 Антон Мидюков 2022-07-12 12:25:01 MSK
(Ответ для Sergey V Turchin на комментарий #9)
> А последним символом ";" будем что-то делать? Я, в принципе, не против и без
> него, т.к. сама возможность уже есть и работает.

Я это предложил, как черновик. Может, у кого-то появятся идеи, как сделать всё более красиво.
Comment 11 manowar@altlinux.org 2022-07-12 13:34:08 MSK
Честно говоря, не до конца улавливаю, зачем это нужно. Чтобы сократить количество desktop-файлов?
Comment 12 Sergey V Turchin 2022-07-12 14:03:39 MSK
(Ответ для manowar@altlinux.org на комментарий #11)
> Честно говоря, не до конца улавливаю, зачем это нужно. Чтобы сократить
> количество desktop-файлов?
Нет. Чтобы увеличить количаство файлов списков, поделив толстые списки покомпонентно по любому желаемому критерию.

Это позволит более гибко складывать группы из "кирпичей". Эти же "кирпичи" можно использовать ещё где-то. Например, в conf.d/distr.mk
@$(call add,LIVE_LISTS,distr/conponent-1)

Т.е. бОльшая гибкость появляется и меньше бардака за счёт меньшего дублирования "кирпичей" в текущих списках.
Comment 13 Антон Мидюков 2022-07-14 13:11:51 MSK
В новом варианте добавил поддержку ';' на конце (удаляется из переменной). Но она должна быть действительно последним символом в строке. Теперь можно указывать на конце ';', а можно не указывать. Пробелы в пути до списка и раньше и тем более сейчас не допускаются.

[#303510] TESTED (try 4) installer-alterator-pkg.git=3.0.1-alt1

Других замечаний никто не высказывал, отправлять в Сизиф?
Comment 14 Sergey V Turchin 2022-07-14 14:15:43 MSK
Я за!
Comment 15 manowar@altlinux.org 2022-07-14 14:58:34 MSK
Я не против.
Comment 16 Repository Robot 2022-07-14 18:35:08 MSK
installer-alterator-pkg-3.0.1-alt1 -> sisyphus:

 Thu Jul 14 2022 Anton Midyukov <antohami@altlinux> 3.0.1-alt1
 - add multiple lists support in X-Alterator-PackageList (Closes: 43186)