Bug 34423

Summary: Добавить в перечисление ("debian", "Ubuntu", "LinuxMint") значение для нашей платформы "altlinux".
Product: Sisyphus Reporter: bliser <bliser>
Component: curaAssignee: Антон Мидюков <antohami>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: antohami
Version: unstable   
Hardware: all   
OS: Linux   
URL: https://forum.altlinux.org/index.php?topic=40708.msg322505#msg322505

Description bliser 2018-01-06 02:10:14 MSK
В файле: "/usr/bin/cura" в строке 15 есть условие if platform.linux_distribution()[0] in ("debian", "Ubuntu", "LinuxMint"). 
     Нужно, чтобы разработчики Cura добавили в перечисление ("debian", "Ubuntu", "LinuxMint") ещё одно значение "altlinux" для нашей платформы. Может быть и второе значение "basealt" - если оно на P8, P9 будет использоваться.
     Это нужно для решения проблемы с запуском приложения на ПК с проприетарным драйвером NVIDIA.
Comment 1 Антон Мидюков 2018-01-06 07:09:02 MSK
Я думаю, что проще просто убрать это условие и выполнять поиск графических библиотек при каждом запуске, раз этот поиск не мешает работе приложения на системах с другими драйверами. Патчик соответствующий сделаю.

Ну, а в апстриме должны заиметь совесть и сделать нормальное условие, которое значится в todo:
# TODO: Needs a "if X11_GFX == 'nvidia'" here. The workaround is only needed on Ubuntu+NVidia drivers. Other drivers are not affected, but fine with this fix.

Проблема эта вылезла сразу при переходе cura на pyQT5. Но разработчики сделали костыль и успокоились...
Comment 2 bliser 2018-01-06 22:34:49 MSK
(In reply to comment #1)
> Я думаю, что проще просто убрать это условие .... . Патчик соответствующий сделаю.

Если так сделать, то версия в репозитории "Sisyphus" станет рабочей, но останется сбой при попытке запустить версию AppImage с сайта разработчика. В Cura (версия AppImage) внести изменение не получится, т.к. там конфигурация в бинарном виде.
Я могу и сам к ним обратится, если вы не против?
Comment 3 Антон Мидюков 2018-01-07 13:58:49 MSK
Я сделал пулл-риквест: https://github.com/Ultimaker/Cura/pull/3097
Исправление уже, кстати, в Сизифе.
Даже, если не примут мой пулл-риквест, то следующая версия всё равно в Альте заработает без патча, так как в список перечисления уже добавлена fedora. А Альт, как выяснилось, прикидывается fedora :-)
Comment 4 Антон Мидюков 2018-01-07 15:59:38 MSK
(В ответ на комментарий №3)
> Я сделал пулл-риквест: https://github.com/Ultimaker/Cura/pull/3097

Пулл-риквест приняли. Ошибку закрываю, так как исправление уже в Сизифе.
Comment 5 bliser 2018-01-07 20:53:17 MSK
(In reply to comment #4)
> Пулл-риквест приняли. Ошибку закрываю, так как исправление уже в Сизифе.

Спасибо. Работоспособность Cura - подтверждаю.