Bug 13126

Summary: [fix 21] Hide desktops in smtube, sane-frontends, mpv, testdisk
Product: [ROSA-based products] ROSA Fresh Reporter: Grigorev Andrey <survolog>
Component: Packages from MainAssignee: ROSA Linux Bugs <bugs>
Status: VERIFIED FIXED QA Contact: ROSA Linux Bugs <bugs>
Severity: normal    
Priority: Normal CC: a.proklov, m.novosyolov, pastordidi, survolog, v.potapov, victorr2007
Version: Plasma5Flags: v.potapov: qa_verified+
a.proklov: published+
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Platform: --- ROSA Vulnerability identifier:
RPM Package: ISO-related:
Bad POT generating: Upstream:
Attachments: smtube

Comment 1 Vladimir Potapov 2023-02-14 05:26:48 MSK 
XScanImage не удалось скрыть, остальное сработало!
Comment 2 Grigorev Andrey 2023-02-14 08:51:47 MSK 
(In reply to Vladimir Potapov from comment #1)
> XScanImage не удалось скрыть, остальное сработало!

sane-frontends 1.0.14-28
https://abf.io/build_lists/4339977
https://abf.io/build_lists/4339975
https://abf.io/build_lists/4339976
https://abf.io/build_lists/4339978
https://abf.io/build_lists/4339979
Comment 4 Grigorev Andrey 2023-02-14 10:40:46 MSK 
(In reply to Grigorev Andrey from comment #2)
> (In reply to Vladimir Potapov from comment #1)
> > XScanImage не удалось скрыть, остальное сработало!
> 
> sane-frontends 1.0.14-28
> https://abf.io/build_lists/4339977
> https://abf.io/build_lists/4339975
> https://abf.io/build_lists/4339976
> https://abf.io/build_lists/4339978
> https://abf.io/build_lists/4339979

sane-frontends 1.0.14-28
https://abf.io/build_lists/4339972
https://abf.io/build_lists/4339970
https://abf.io/build_lists/4339971
https://abf.io/build_lists/4339973
https://abf.io/build_lists/4339974
Comment 6 Mikhail Novosyolov 2023-02-18 17:22:58 MSK 
Долбанулись что ли?mpv нормальная программа, зачем ее прятать??? Вырежьте ее из зависимостей, и все, что вы ломаете нормальный пакет! Плееры, который основаны на mpv, зависят от libmpv, а не от самого mpv! Я, например, иногда пользуюсь самим mpv, как плеером, с какого перепуга его desktop должен быть скрыт. Что за костыли необдуманные опять!
Comment 7 Mikhail Novosyolov 2023-02-18 17:24:03 MSK 
smtube аналогично
Comment 8 Mikhail Novosyolov 2023-02-18 17:31:52 MSK 
(In reply to Vladimir Potapov from comment #5)
> smtube 21.10.0-3
> https://abf.io/build_lists/4339747
> https://abf.io/build_lists/4339745
> https://abf.io/build_lists/4339746
> https://abf.io/build_lists/4339748
> 
> mpv 0.34.1-8
> https://abf.io/build_lists/4339774
> https://abf.io/build_lists/4339772
> https://abf.io/build_lists/4339773
> https://abf.io/build_lists/4339775
> 

Прошу не ломать репозиторий необдуманными костылями. 
Сборки отклонил, git откатил.
При добавлении подобных костылей в git давайте уважать друг друга и вставлять пояснения, что это за дичь, зачем она нужна.
Уберите smtube, mpv из зависимостей, где они не нужны, и не ломайте пакеты.
Такой подход к работе недопустим.

И прошу QA не подталкивать разработчиков к подобным действиям!
Виктор, посмотри, пожалуйста, почему господа костыльщики так возбудились, если smtube и mpv мешают в KDE, то значит их оттуда нужно убрать по-нормальному, а не сломав пакеты.
Comment 9 Mikhail Novosyolov 2023-02-18 17:50:54 MSK 
smplayer "Requires" mpv, уверен, что это ошибочная зависимость, а на самом деле нужна просто libmpv, зависимость от которой rpm проставляет автоматически.

Что касается smtube, если /usr/bin/smtube кому-то нужен, то можно сделать подпакет smtube-core со всеми файлами, кроме desktop, а в smtube оставить desktop и зависимость от -core.

smplayer сейчас сам сделаю, по smtube уточните суть проблемы.
Comment 10 Mikhail Novosyolov 2023-02-18 17:55:21 MSK 
Попробовал, smplayer без пакета mpv не работает, надо смотреть, что именно нужно, если сам /usr/bin/mpv, то можно сделать подпакет mpv-core.
Comment 11 Dmitry Postnikov 2023-02-18 23:45:52 MSK 
SMplayer теперь фронтэнд для Mplayer и MPV

https://www.smplayer.info/ru/mpv

По-этому ему MPV в полном объеме нужен.
Не понятно, зачем MPV "вырезать" из меню, и все остальное тоже.
Comment 12 Dmitry Postnikov 2023-02-18 23:52:32 MSK 
Тут 129я строка, написано: 

Remove smtube

https://github.com/smplayer-dev/smplayer/blob/master/smplayer.spec

Т.е. они вырезали smtube.
Comment 13 VictorR2007 2023-02-19 01:59:25 MSK 
(In reply to Dmitry Postnikov from comment #12)
> Тут 129я строка, написано: 
> 
> Remove smtube
> 
> https://github.com/smplayer-dev/smplayer/blob/master/smplayer.spec
> 
> Т.е. они вырезали smtube.

Ну он не обязателен для работы smplayer, как mpv.
Поэтому и не его в спеке.
А в настройках самого smtube, вроде есть настройка,
какой именно проигрыватель вызывать по умолчанию,
для запуска воспроизведения.
Comment 14 Vladimir Potapov 2023-02-20 10:07:24 MSK 
(In reply to Dmitry Postnikov from comment #11)
> Не понятно, зачем MPV "вырезать" из меню, и все остальное тоже.
Зачем мне несколько интерфейсов к одному и тому же плееру или сканеру?
Comment 15 VictorR2007 2023-02-20 10:36:52 MSK 
(In reply to Vladimir Potapov from comment #14)
> (In reply to Dmitry Postnikov from comment #11)
> > Не понятно, зачем MPV "вырезать" из меню, и все остальное тоже.
> Зачем мне несколько интерфейсов к одному и тому же плееру или сканеру?

Ну да, очень плохо поступили с mpv.
Скрыли его из контекстного меню открытия файлов.
Это единственный проигрыватель, я про голый mpv,
который не ведёт историю открытых файлов.
В остальных проигрывателях есть список воспроизведения,
и если открыть файл с помощь smplayer, то список изменится.
Например, я, открываю единичные видеофайлы с помощью mpv,
а в smplayer у меня список воспроизведения, настроенный 
на продолжение воспроизведения, и в котором у меня 
настроен просмотр многосерийного фильма или сериала.
Открытие любого видеофайла из контекстного меню с помощью 
smplayer закрывает этот список и теряется очередь воспроизведения.
C smtube тоже плохо поступили.
Не нравится, так и уберите его из образа, зачем портить.
Отличное приложение, с категориями топов на панели.
Очень удобно открыть приложение, и посмотреть популярные 
видео из нужной категории, например, «Животные».
Напишу не форуме, как разблокировать эти приложения.
Comment 16 VictorR2007 2023-02-20 10:41:40 MSK 
SMTube, это не дополнение к SMPlayer, а отдельное приложение.
Перейдя в настройки которого, в меню Вид - Настройки - Плееры,
можно выбрать проигрыватель, который будет воспроизводить видео.
Comment 17 Vladimir Potapov 2023-02-20 10:43:20 MSK 
(In reply to VictorR2007 from comment #15)
> (In reply to Vladimir Potapov from comment #14)
> > (In reply to Dmitry Postnikov from comment #11)
> > > Не понятно, зачем MPV "вырезать" из меню, и все остальное тоже.
> > Зачем мне несколько интерфейсов к одному и тому же плееру или сканеру?
> 
> Ну да, очень плохо поступили с mpv.
> Скрыли его из контекстного меню открытия файлов.
Ну вот там он нужен, а в обычном меню - зачем? Можно ли сделать, чтоб он в контекстном был, а в общем-нет?

> C smtube тоже плохо поступили.
> Не нравится, так и уберите его из образа, зачем портить
Его smplayer тащит в рекомендуемых. Убрать?

> Напишу не форуме, как разблокировать эти приложения.
Не надо, Миша ж откатил и непонятно, что будет.
Comment 18 Dmitry Postnikov 2023-02-20 10:46:44 MSK 
(In reply to Vladimir Potapov from comment #14)
> (In reply to Dmitry Postnikov from comment #11)
> > Не понятно, зачем MPV "вырезать" из меню, и все остальное тоже.
> Зачем мне несколько интерфейсов к одному и тому же плееру или сканеру?

Ладно, всё, я понял. Заканчиваем разговор. Что сделали, то сделали.
Comment 19 Vladimir Potapov 2023-02-20 10:49:53 MSK 
(In reply to Dmitry Postnikov from comment #18)
> Ладно, всё, я понял. Заканчиваем разговор. Что сделали, то сделали.
В спорах рождается истина. mpv действительно хорош для контекстного меню, если есть возможность его оставить именно там - будет прекрасно!
А по множественным иксовым интерфейсам для сканеров, как я понимаю, возражений нет?
Comment 20 VictorR2007 2023-02-20 11:01:28 MSK 
(In reply to Vladimir Potapov from comment #17)
> Не надо, Миша ж откатил и непонятно, что будет.

Он откатил изменения, но пакеты опубликованы уже в testing.
Comment 21 Vladimir Potapov 2023-02-20 11:17:29 MSK 
(In reply to VictorR2007 from comment #20)
> (In reply to Vladimir Potapov from comment #17)
> > Не надо, Миша ж откатил и непонятно, что будет.
> 
> Он откатил изменения, но пакеты опубликованы уже в testing.
И? Я теперь не имею права их одобрить, только отклонить.
Comment 22 Mikhail Novosyolov 2023-02-20 11:20:14 MSK 
Ожидаемое поведение от любого пакета с графическим интерфейсом — наличие и в меню приложений,и  в меню "Открыть с помощью", а иначе сделать даже невозможно.

Сейчас нужно выяснить, есть ли в smplayer возможность вызывать smtube.
Comment 23 VictorR2007 2023-02-20 11:24:02 MSK 
Created attachment 5863 [details]
smtube

(In reply to Mikhail Novosyolov from comment #22)
> 
> Сейчас нужно выяснить, есть ли в smplayer возможность вызывать smtube.
Умеет.
Comment 24 Mikhail Novosyolov 2023-02-20 11:27:10 MSK 
(In reply to VictorR2007 from comment #23)
> Created attachment 5863 [details]
> smtube
> 
> (In reply to Mikhail Novosyolov from comment #22)
> > 
> > Сейчас нужно выяснить, есть ли в smplayer возможность вызывать smtube.
> Умеет.

Тогда сделаю сейчас выделение подпакетов в mpv и smtube в личном репозитории и покажу.
Comment 25 Vladimir Potapov 2023-02-20 11:31:59 MSK 
(In reply to Mikhail Novosyolov from comment #24)
> Тогда сделаю сейчас выделение подпакетов в mpv и smtube в личном репозитории
> и покажу.
Да, посмотрел, smtube вызывается из меню smplayer. Тогда давайте его оставим. 
Еще один интерфейс к mpv же выглядит излишним, что он умеет, чего не умеет smplayer?
Comment 26 Mikhail Novosyolov 2023-02-20 11:41:13 MSK 
(In reply to Vladimir Potapov from comment #25)
> (In reply to Mikhail Novosyolov from comment #24)
> > Тогда сделаю сейчас выделение подпакетов в mpv и smtube в личном репозитории
> > и покажу.
> Да, посмотрел, smtube вызывается из меню smplayer. Тогда давайте его
> оставим. 
Можно все равно путем разбиения на подпакеты сделать так, чтобы /usr/bin/smtube подтягивался, а /usr/share/applications/smtube.desktop — нет.

> Еще один интерфейс к mpv же выглядит излишним, что он умеет, чего не умеет
> smplayer?
Да установка одного плеера не должна подтягивать другой, путем разбивки на подпакеты его ярлык можно убрать.
Comment 27 Mikhail Novosyolov 2023-02-20 17:31:58 MSK 
Сделал разделение в mpv
https://abf.io/mikhailnov/mpv/commit/7636e5e0675c5cdc6c8a1ed20ff7f75df7008153
Посмотрите, не будет ли идей
Comment 28 Vladimir Potapov 2023-02-21 08:58:17 MSK 
(In reply to Mikhail Novosyolov from comment #27)
> Сделал разделение в mpv
> https://abf.io/mikhailnov/mpv/commit/7636e5e0675c5cdc6c8a1ed20ff7f75df7008153
> Посмотрите, не будет ли идей
идея только "полтора плеера", оставить в контекстном меню, убрав из главного. Если это неприемлемо - собирай так.
Comment 29 Mikhail Novosyolov 2023-02-22 01:51:51 MSK 
mpv
https://abf.io/build_lists/4361217
https://abf.io/build_lists/4361219
https://abf.io/build_lists/4361215
https://abf.io/build_lists/4361223
https://abf.io/build_lists/4361222

smplayer
https://abf.io/build_lists/4361214
https://abf.io/build_lists/4361216
https://abf.io/build_lists/4361218
https://abf.io/build_lists/4361220
https://abf.io/build_lists/4361221

Ожидаемое поведение:
* существующие установки обновляются с сохранением функционала
* новые установки smplayer не подтягивают ярлыки mpv
Comment 30 Mikhail Novosyolov 2023-02-22 12:01:12 MSK 
Ошибся. Переделаю.
Comment 32 Mikhail Novosyolov 2023-02-22 16:25:46 MSK 
Образ plasma5 с тестингами, mpv, smplayer: https://abf.io/platforms/rosa2021.1/products/318/product_build_lists/45780
Comment 35 Vladimir Potapov 2023-02-27 09:53:29 MSK 
https://abf.io/build_lists/4361221
ошибка сборки
Comment 36 Mikhail Novosyolov 2023-02-27 10:43:37 MSK 
(In reply to Vladimir Potapov from comment #35)
> https://abf.io/build_lists/4361221
> ошибка сборки

Это RISC-V, забьем пока