Bug 39632 - Исправление процесса копирования locale в efi-grub
Summary: Исправление процесса копирования locale в efi-grub
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: mkimage-profiles (show other bugs)
Version: unstable
Hardware: all Linux
: P5 normal
Assignee: Антон Мидюков
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-01 15:02 MSK by underwit
Modified: 2021-02-16 09:46 MSK (History)
3 users (show)

See Also:

Attachments
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 underwit 2021-02-01 15:02:25 MSK 
Добрый день!
Прошу принять изменения.

Копируются переводы только для языков указанных в grub.cfg

http://git.altlinux.org/people/underwit/packages/?p=mkimage.git;a=commit;h=a2482e2dc7bc9accb768ce0b3e8f4cf15f53676a

Спасибо.
Comment 1 Sergey V Turchin 2021-02-01 15:13:39 MSK 
А разве в mkimage-profiles предварительно нельзя удалить ненужные переводы grub?
Comment 2 Антон Мидюков 2021-02-01 15:33:40 MSK 
(Ответ для Sergey V Turchin на комментарий #1)
> А разве в mkimage-profiles предварительно нельзя удалить ненужные переводы
> grub?

Нет. Никаких скриптов из mkimage-profiles не выполнить внутри цели tools/mki-pack-boot. Нет возможности врезаться в работу этого скрипта. Внутри него выполняется mki-copy-efiboot, затем mki-pack-isoboot, который делает iso.
Comment 3 Sergey V Turchin 2021-02-01 16:40:02 MSK 
Просто, список языков слишком захардкоден как-то.

Если бы какая-то переменная была, которую, например, mkimage-profiles смогут переопределять...
Comment 4 Антон Мидюков 2021-02-01 16:47:09 MSK 
(Ответ для Sergey V Turchin на комментарий #3)
> Просто, список языков слишком захардкоден как-то.
> 
> Если бы какая-то переменная была, которую, например, mkimage-profiles смогут
> переопределять...

Так ничего не мешает переопределять список в mkimage-profiles. Только добавлять в переменную значения вида ru_RU=Russian не очень нравится. Хотя, можно и так...
Comment 5 Sergey V Turchin 2021-02-01 16:55:52 MSK 
(Ответ для Антон Мидюков на комментарий #4)
> Так ничего не мешает переопределять список в mkimage-profiles.
Пока список никто со стороны не парсит, естественно не мешает.
Comment 6 Sergey V Turchin 2021-02-02 10:16:44 MSK 
Т.е. плохо, когда формат списка в одном месте, а парсилка в другом.
Comment 7 Антон Мидюков 2021-02-10 20:25:56 MSK 
(Ответ для Sergey V Turchin на комментарий #6)
> Т.е. плохо, когда формат списка в одном месте, а парсилка в другом.

Тогда копирование нужных локалей стоит осуществлять в mkimage-profiles в features.in/grub/stage1/scripts.d/01-grub или соседнем с ним скрипте (создать). Только нужно будет в STAGE1_PACKAGES добавить grub-common.
Comment 8 Антон Мидюков 2021-02-12 22:23:01 MSK 
Я сделаю(Ответ для Антон Мидюков на комментарий #7)
> (Ответ для Sergey V Turchin на комментарий #6)
> > Т.е. плохо, когда формат списка в одном месте, а парсилка в другом.
> 
> Тогда копирование нужных локалей стоит осуществлять в mkimage-profiles в
> features.in/grub/stage1/scripts.d/01-grub или соседнем с ним скрипте
> (создать). Только нужно будет в STAGE1_PACKAGES добавить grub-common.

Попробовал, получилось. Будет в mkimage-profiles.
Comment 9 Антон Мидюков 2021-02-16 09:37:48 MSK 
(Ответ для Антон Мидюков на комментарий #8)
> Я сделаю(Ответ для Антон Мидюков на комментарий #7)
> > (Ответ для Sergey V Turchin на комментарий #6)
> > > Т.е. плохо, когда формат списка в одном месте, а парсилка в другом.
> > 
> > Тогда копирование нужных локалей стоит осуществлять в mkimage-profiles в
> > features.in/grub/stage1/scripts.d/01-grub или соседнем с ним скрипте
> > (создать). Только нужно будет в STAGE1_PACKAGES добавить grub-common.
> 
> Попробовал, получилось. Будет в mkimage-profiles.

Сделано в mkimage-profiles 1.4.7-alt1