Bug 33906 - не запускается без /etc/os-release
Summary: не запускается без /etc/os-release
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: connector (show other bugs)
Version: unstable
Hardware: all Linux
: P3 minor
Assignee: ekorneechev@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 35595
  Show dependency tree
 
Reported: 2017-09-18 12:26 MSK by Michael Shigorin
Modified: 2018-11-12 13:15 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2017-09-18 12:26:01 MSK
elbrus ~> connector 
grep: /etc/os-release: No such file or directory
Traceback (most recent call last):
  File "/usr/bin/connector", line 6, in <module>
    import gui
  File "/usr/share/connector/gui.py", line 8, in <module>
    from ctor import *
  File "/usr/share/connector/ctor.py", line 4, in <module>
    import time, properties
  File "/usr/share/connector/properties.py", line 9, in <module>
    from GLOBAL import *
  File "/usr/share/connector/GLOBAL.py", line 49, in <module>
    OS = subprocess.check_output("grep '^ID=' /etc/os-release; exit 0",shell=True, universal_newlines=True).strip().split('=')[1]
IndexError: list index out of range
elbrus ~> cat /etc/os-release
cat: /etc/os-release: No such file or directory
elbrus ~> rpm -q connector
connector-1.5.1-alt2
elbrus ~> _
Comment 1 Evgeniy Korneechev 2017-09-19 09:05:41 MSK
Код программы один как для Альта, так и для Mint/Ubuntu. После считывания ID из /etc/os-release, определяются ОС-зависимые глобальные переменные.

+Requires: /etc/os-release
или лучше при заведомом отсутствии файла считать, что это Альт?)
Comment 2 Michael Shigorin 2017-09-19 16:23:15 MSK
(В ответ на комментарий №1)
> +Requires: /etc/os-release
Можно и так, конечно, но субъективно перебор.

> или лучше при заведомом отсутствии файла считать, что это Альт?)
Ммм... можно ещё проверить /etc/altlinux-release и если есть -- тогда альт :)
Comment 3 Evgeniy Korneechev 2017-09-19 17:08:22 MSK
(В ответ на комментарий №2)
> Ммм... можно ещё проверить /etc/altlinux-release и если есть -- тогда альт :)
А он обязательно присутствует во всех дистрибутивах (альта)?
Оба файла принадлежат одному и тому же пакету branding-*
Comment 4 Anton Farygin 2017-09-19 17:31:35 MSK
os-release нет в hasher и это ошибка, она у нас даже висит где-то в bugzilla
Comment 5 Anton Farygin 2017-09-19 17:34:10 MSK
почему ошибка - на наличие os-release закладывается очень много проектов, это уже стандарт де-факто.

https://www.freedesktop.org/software/systemd/man/os-release.html
Comment 6 Evgeniy Korneechev 2017-09-19 17:55:14 MSK
(В ответ на комментарий №4)
> os-release нет в hasher и это ошибка, она у нас даже висит где-то в bugzilla
http://bugs.altlinux.org/33656 из той же области?
Comment 7 Michael Shigorin 2017-09-19 17:59:31 MSK
(В ответ на комментарий №3)
> > можно ещё проверить /etc/altlinux-release
> А он обязательно присутствует во всех дистрибутивах (альта)?
Да.

> Оба файла принадлежат одному и тому же пакету branding-*
/etc/altlinux-release есть в любом известном мне поставщике altlinux-release, который требуется пакету basesystem; для /etc/os-release это не так.
Comment 8 Repository Robot 2017-09-20 11:41:02 MSK
connector-1.5.2-alt1 -> sisyphus:

Wed Sep 20 2017 Evgeniy Korneechev <ekorneechev@altlinux.org> 1.5.2-alt1
- OS check updated (ALT #33906)
- Added notifications for mode 'kiosk'
Comment 9 Michael Shigorin 2017-09-21 12:44:48 MSK
Спасибо, работает :)