Bug 49863 - Удалите требование firefox для noarch пакета
Summary: Удалите требование firefox для noarch пакета
Status: NEW
Alias: None
Product: Sisyphus
Classification: Development
Component: gnome-kiosk-search-appliance (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 blocker
Assignee: Yuri N. Sedunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 46625
  Show dependency tree
 
Reported: 2024-04-02 12:38 MSK by Ajrat Makhmutov
Modified: 2024-04-27 15:33 MSK (History)
9 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ajrat Makhmutov 2024-04-02 12:38:36 MSK
Удаляю firefox-unmet-holder из репозитория.
Прошу заапрувить задание https://packages.altlinux.org/ru/tasks/343742/
Comment 1 Sergey V Turchin 2024-04-02 13:28:51 MSK
(Ответ для Ajrat Makhmutov на комментарий #0)
> Удаляю firefox-unmet-holder из репозитория.
Согласовано с мантнейнером firefox.
Comment 2 Sergey V Turchin 2024-04-09 15:34:36 MSK
issue#131733
Comment 3 Sergey V Turchin 2024-04-10 13:34:54 MSK
Надеюсь, директора на баг подписывать не придётся.
Comment 4 AEN 2024-04-10 23:35:11 MSK
Прошу объяснить, зачем нужно удалять firefox-unmet-holder. Кому он помешал?
Без эмоций.
Comment 5 Ajrat Makhmutov 2024-04-11 10:53:26 MSK
(Ответ для AEN на комментарий #4)
> Прошу объяснить, зачем нужно удалять firefox-unmet-holder. Кому он помешал?
> Без эмоций.
Пакет никому не помешал. Это "костыль", который я хочу убрать.
Он возник из-за того, что изначально firefox собирался на всех архитектурах и на него насобирали noarch аддонов, которые его требовали, как и gnome-kiosk-search-appliance. Но сейчас аддоны собирают без требования firefox'а, а те, что требовали - исправили (firefox-r-kiosk например).
Сейчас есть всего 3 unmet'а которые нужно решить: подпакет firefox'а, аддон который никто не мейнтейнит с 2012-го и gnome-kiosk-search-appliance:
https://git.altlinux.org/tasks/343742/logs/events.5.1.log
Из неприятного: мне, как новому мейнтейнеру firefox'а, нужно не забывать вместе с ним обновлять unmet-holder и пользователь на ppc64le или i586 может прописать "apt-get install firefox" и установить абсолютно пустой пакет.
Comment 6 AEN 2024-04-11 11:44:06 MSK
(Ответ для Ajrat Makhmutov на комментарий #5)
> (Ответ для AEN на комментарий #4)
> > Прошу объяснить, зачем нужно удалять firefox-unmet-holder. Кому он помешал?
> > Без эмоций.
> Пакет никому не помешал. Это "костыль", который я хочу убрать.
> Он возник из-за того, что изначально firefox собирался на всех архитектурах
> и на него насобирали noarch аддонов, которые его требовали, как и
> gnome-kiosk-search-appliance. Но сейчас аддоны собирают без требования
> firefox'а, а те, что требовали - исправили (firefox-r-kiosk например).
> Сейчас есть всего 3 unmet'а которые нужно решить: подпакет firefox'а, аддон
> который никто не мейнтейнит с 2012-го и gnome-kiosk-search-appliance:
> https://git.altlinux.org/tasks/343742/logs/events.5.1.log
> Из неприятного: мне, как новому мейнтейнеру firefox'а, нужно не забывать
> вместе с ним обновлять unmet-holder и пользователь на ppc64le или i586 может
> прописать "apt-get install firefox" и установить абсолютно пустой пакет.

Айрат, спасибо.
Прошу Вас в дальнейшем давать в тексте баги подобные технические разъяснения.
Сейчас мы готовимся к бранчеванию. Задача не срочная, но прошу Юрия и Ивана оценить, нет ли рисков удаления этого пакета, в том числе для новых архитектур.
Comment 7 Sergey V Turchin 2024-04-11 12:10:42 MSK
(Ответ для Ajrat Makhmutov на комментарий #5)
> Из неприятного: мне, как новому мейнтейнеру firefox'а, нужно не забывать
> вместе с ним обновлять unmet-holder
В случае, если не удастся выкинуть firefox-unmet-holder, нужно будет добавить в firefox.spec сборку пустого пакета для любых остальных архитектур.
Comment 8 Sergey V Turchin 2024-04-11 12:11:46 MSK
> В случае, если не удастся выкинуть firefox-unmet-holder, нужно будет
> добавить в firefox.spec сборку пустого пакета для любых остальных архитектур.
Вплоть до упаковки туда фейкового /usr/bin/firefox.
Comment 9 Ivan A. Melnikov 2024-04-11 12:15:00 MSK
На riscv64 и loongarch64 firefox есть и поддерживается, так что никаких рисков от удаления firefox-unmet-holder я не вижу.
Comment 10 Ajrat Makhmutov 2024-04-25 17:12:09 MSK
Считаю, что нужно удалить firefox-unmet-holder до выхода p11
Comment 11 Ajrat Makhmutov 2024-04-25 17:30:44 MSK
Оформил новый релиз по NMU Policy
https://packages.altlinux.org/ru/tasks/343742/
Comment 12 Andrey Cherepanov 2024-04-26 11:11:02 MSK
Юрий, для использования браузера, может, лучше прописать зависимость в search-appliance не на fiefox явно, а на /usr/bin/xbrowser? Думаю, это решит проблему.
Comment 13 Yuri N. Sedunov 2024-04-26 11:15:48 MSK
(Ответ для Andrey Cherepanov на комментарий #12)
> Юрий, для использования браузера, может, лучше прописать зависимость в
> search-appliance не на fiefox явно, а на /usr/bin/xbrowser? Думаю, это решит
> проблему.

$ grep firefox `rpmql gnome-kiosk-search-appliance`
/usr/share/applications/org.gnome.Kiosk.SearchApp.desktop:Exec=/usr/bin/firefox --kiosk --private-window --new-instance  https://www.ya.ru
Comment 14 Sergey V Turchin 2024-04-26 11:20:15 MSK
(Ответ для Sergey V Turchin на комментарий #8)
> > добавить в firefox.spec сборку пустого пакета для любых остальных архитектур.
> Вплоть до упаковки туда фейкового /usr/bin/firefox.
Видимо, придётся идти этим путём. Упаковать /usr/bin/firefox символической ссылкой на /bin/true для любых архитектур, где он не собирается в нормальном виде.
Comment 15 Andrey Cherepanov 2024-04-26 12:23:04 MSK
(Ответ для Yuri N. Sedunov на комментарий #13)
> (Ответ для Andrey Cherepanov на комментарий #12)
> > Юрий, для использования браузера, может, лучше прописать зависимость в
> > search-appliance не на fiefox явно, а на /usr/bin/xbrowser? Думаю, это решит
> > проблему.
> 
> $ grep firefox `rpmql gnome-kiosk-search-appliance`
> /usr/share/applications/org.gnome.Kiosk.SearchApp.desktop:Exec=/usr/bin/
> firefox --kiosk --private-window --new-instance  https://www.ya.ru

Работает и xbrowser --kiosk --private-window --new-instance  https://www.ya.ru
Comment 16 Yuri N. Sedunov 2024-04-26 13:10:57 MSK
$ grep /usr/bin/xbrowser /ALT/Sisyphus/ppc64le/base/contents_index 
$
Comment 17 Sergey V Turchin 2024-04-26 13:19:39 MSK
(Ответ для Yuri N. Sedunov на комментарий #16)
> $ grep /usr/bin/xbrowser /ALT/Sisyphus/ppc64le/base/contents_index 
> $
apt-get install /usr/bin/xbrowser
Comment 18 Sergey V Turchin 2024-04-26 13:26:48 MSK
(Ответ для Yuri N. Sedunov на комментарий #16)
> $ grep /usr/bin/xbrowser /ALT/Sisyphus/ppc64le/base/contents_index 
Ну, или 
grep /usr/bin/firefox /ALT/Sisyphus/ppc64le/base/contents_index
Comment 19 Andrey Cherepanov 2024-04-26 13:53:33 MSK
(Ответ для Yuri N. Sedunov на комментарий #16)
> $ grep /usr/bin/xbrowser /ALT/Sisyphus/ppc64le/base/contents_index 
> $

Не вопрос. Тогда сделать подпакет не noarch и собирать только для архитектур, под которые собран firefox.
Comment 20 Ajrat Makhmutov 2024-04-26 14:09:58 MSK
(Ответ для Andrey Cherepanov на комментарий #19)
> Не вопрос. Тогда сделать подпакет не noarch и собирать только для
> архитектур, под которые собран firefox.

Поддерживаю. Предлагаю сделать новый макрос в rpm-build-firefox
"%firefox_arches x86_64 armh aarch64" и использовать его так:
"ExclusiveArch: %firefox_arches", по аналогии c %go_arches.
Comment 21 Andrey Cherepanov 2024-04-26 15:45:47 MSK
(Ответ для Ajrat Makhmutov на комментарий #20)
> (Ответ для Andrey Cherepanov на комментарий #19)
> > Не вопрос. Тогда сделать подпакет не noarch и собирать только для
> > архитектур, под которые собран firefox.
> 
> Поддерживаю. Предлагаю сделать новый макрос в rpm-build-firefox
> "%firefox_arches x86_64 armh aarch64" и использовать его так:
> "ExclusiveArch: %firefox_arches", по аналогии c %go_arches.

Сделайте.
Comment 22 Ivan A. Melnikov 2024-04-26 16:30:25 MSK
(In reply to Ajrat Makhmutov from comment #20)
> Поддерживаю. Предлагаю сделать новый макрос в rpm-build-firefox
> "%firefox_arches x86_64 armh aarch64"

Пожалуйста, не забывайте, что у нас есть ещё, как минимум, %e2k, loongarch64 и riscv64, и на всех них firefox есть.

Предлагаю всё-таки делать %no_firefox_arches и использовать ExcludeArch.
Comment 23 Ajrat Makhmutov 2024-04-26 16:38:43 MSK
(Ответ для Ivan A. Melnikov на комментарий #22)
> Предлагаю всё-таки делать %no_firefox_arches и использовать ExcludeArch.

Ок.
Comment 24 Sergey V Turchin 2024-04-26 16:40:26 MSK
(Ответ для Ivan A. Melnikov на комментарий #22)
> Предлагаю всё-таки делать %no_firefox_arches и использовать ExcludeArch.
👍
Comment 25 Ajrat Makhmutov 2024-04-27 13:51:07 MSK
(Ответ для Andrey Cherepanov на комментарий #19)
> 
> Не вопрос. Тогда сделать подпакет не noarch и собирать только для
> архитектур, под которые собран firefox.

Видимо, собирать пакет на всех архитектурах и его подпакет на определённых невозможно: https://git.altlinux.org/tasks/343742/logs/events.7.1.log
Но я выяснил, что firefox в этом пакете выступает только в качестве примера полноэкранного приложения:
https://git.altlinux.org/srpms/g/gnome-kiosk.git?p=gnome-kiosk.git;a=blob;f=gnome-kiosk/search-app/org.gnome.Kiosk.SearchApp.desktop.in.in
https://gitlab.gnome.org/GNOME/gnome-kiosk/#sample-application
То есть для работы search-appliance firefox не требуется и вместо него можно поставить любое полноэкранное приложение. А можно вообще удалить этот пример (/usr/share/applications/org.gnome.Kiosk.SearchApp.desktop) и пусть пользователь сам пишет с нуля.
Comment 26 Sergey V Turchin 2024-04-27 14:52:49 MSK
(Ответ для Ajrat Makhmutov на комментарий #25)
> Видимо, собирать пакет на всех архитектурах и его подпакет на определённых
> невозможно: https://git.altlinux.org/tasks/343742/logs/events.7.1.log
Возможно. Просто gnome-kiosk надо так же.

[...]
> А можно вообще удалить этот пример
Да. Было бы неплохо. Или перенести в документацию.
Тогда проблема пропадёт.
Comment 27 Ajrat Makhmutov 2024-04-27 15:33:39 MSK
Весь подпакет gnome-kiosk-search-appliance является примером с браузером. Если из него вырезать файл /usr/share/applications/org.gnome.Kiosk.SearchApp.desktop, то он вообще потеряет свою ценность.
Юрий, предлагаю просто удалить подпакет search-appliance.