[devel] linux_distribution() из Python

[devel] linux_distribution() из Python

[Andrey Cherepanov]

Предлагается в функции linux_distribution() из Python 2.x и 3.x
возвращать первым параметром "alt" для любых дистрибутивов с
/etc/altlinux-release.

Сейчас оно берёт первое слово из /etc/altlinux-release, что приводит к
https://bugzilla.altlinux.org/show_bug.cgi?id=34266

Возражения есть?

-- 
Andrey Cherepanov
cas@altlinux.org

Re: [devel] linux_distribution() из Python

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 271 bytes --]

On Thu, May 03, 2018 at 04:41:59PM +0300, Andrey Cherepanov wrote:
> Предлагается в функции linux_distribution() из Python 2.x и 3.x
> возвращать первым параметром "alt" для любых дистрибутивов с
> /etc/altlinux-release.

А там регистр важен или нет?


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] linux_distribution() из Python

[Andrey Cherepanov]

03.05.2018 16:53, Dmitry V. Levin пишет:
> On Thu, May 03, 2018 at 04:41:59PM +0300, Andrey Cherepanov wrote:
>> Предлагается в функции linux_distribution() из Python 2.x и 3.x
>> возвращать первым параметром "alt" для любых дистрибутивов с
>> /etc/altlinux-release.
> А там регистр важен или нет?

А это смотря для кого. ceph и ceph-deploy принудительно distroname
переводят в нижний регистр. А для поиска использующих эту функцию я и
написал.

-- 
Andrey Cherepanov
cas@altlinux.org

Re: [devel] linux_distribution() из Python

[Anton Farygin]

03.05.2018 16:41, Andrey Cherepanov пишет:
> Предлагается в функции linux_distribution() из Python 2.x и 3.x
> возвращать первым параметром "alt" для любых дистрибутивов с
> /etc/altlinux-release.
>
> Сейчас оно берёт первое слово из /etc/altlinux-release, что приводит к
> https://bugzilla.altlinux.org/show_bug.cgi?id=34266
>
> Возражения есть?
>
Твоё предложение выглядит не очень здраво и может кому-то что-то сломать.

Вот тут есть ещё одна реализация для тех же целей:

http://git.altlinux.org/tasks/205194/gears/400/git?p=git;a=commitdiff;h=330fa74425e6728803517bbd84348c0c2b7d921f

Re: [devel] linux_distribution() из Python

[Andrey Cherepanov]

03.05.2018 17:40, Anton Farygin пишет:
> 03.05.2018 16:41, Andrey Cherepanov пишет:
>> Предлагается в функции linux_distribution() из Python 2.x и 3.x
>> возвращать первым параметром "alt" для любых дистрибутивов с
>> /etc/altlinux-release.
>>
>> Сейчас оно берёт первое слово из /etc/altlinux-release, что приводит к
>> https://bugzilla.altlinux.org/show_bug.cgi?id=34266
>>
>> Возражения есть?
>>
> Твоё предложение выглядит не очень здраво и может кому-то что-то сломать.
>
> Вот тут есть ещё одна реализация для тех же целей:
>
> http://git.altlinux.org/tasks/205194/gears/400/git?p=git;a=commitdiff;h=330fa74425e6728803517bbd84348c0c2b7d921f
>
>
Проверь на branding-alt-sisyphus-release.
И зачем плодить дополнительное API и кто им будет пользоваться?

-- 
Andrey Cherepanov
cas@altlinux.org

Re: [devel] linux_distribution() из Python

[Sergey V Turchin]

On Thursday, 3 May 2018 17:52:18 MSK Andrey Cherepanov wrote:

[...]
> > http://git.altlinux.org/tasks/205194/gears/400/git?p=git;a=commitdiff;h=33
> > 0fa74425e6728803517bbd84348c0c2b7d921f
> Проверь на branding-alt-sisyphus-release.
Достаточно его почитать. Там то же, что и в других *-release в отличие от...

> И зачем плодить дополнительное API
Чтобы такого бардака не было.

> и кто им будет пользоваться?
Все. Это просто.

-- 
Regards, Sergey.

Re: [devel] linux_distribution() из Python

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 736 bytes --]

On Thu, May 03, 2018 at 05:40:44PM +0300, Anton Farygin wrote:
> 03.05.2018 16:41, Andrey Cherepanov пишет:
> > Предлагается в функции linux_distribution() из Python 2.x и 3.x
> > возвращать первым параметром "alt" для любых дистрибутивов с
> > /etc/altlinux-release.
> >
> > Сейчас оно берёт первое слово из /etc/altlinux-release, что приводит к
> > https://bugzilla.altlinux.org/show_bug.cgi?id=34266
> >
> > Возражения есть?
> >
> Твоё предложение выглядит не очень здраво и может кому-то что-то сломать.
> 
> Вот тут есть ещё одна реализация для тех же целей:
> 
> http://git.altlinux.org/tasks/205194/gears/400/git?p=git;a=commitdiff;h=330fa74425e6728803517bbd84348c0c2b7d921f

А что она реализует?


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] linux_distribution() из Python

[Anton Farygin]

https://www.freedesktop.org/software/systemd/man/os-release.html

$ cat /etc/os-release
NAME="ALT"
VERSION="8.2 "
ID=altlinux
VERSION_ID=8.2
PRETTY_NAME="ALT Workstation K 8.2  (Centaurea Ruthenica)"
ANSI_COLOR="1;33"
CPE_NAME="cpe:/o:alt:kworkstation:8.2"
HOME_URL="http://www.basealt.ru"
BUG_REPORT_URL="https://bugs.altlinux.org/"

Если я правильно понимаю, что с версии 8.0 у нас этот файл всегда 
корректно заполнен.


04.05.2018 14:15, Andrey Bychkov пишет:
>
> Использует файл идентификации /etc/os-release (если он имеется) для 
> полочения информации о дистрибутиве. Парсит поля файла, а именно NAME, 
> VERSION_ID, ID. Если нет, то информация извлекается старым способом.
>
>> On Thu, May 03, 2018 at 05:40:44PM +0300, Anton Farygin wrote:
>>> 03.05.2018 16:41, Andrey Cherepanov пишет:
>>>> Предлагается в функции linux_distribution() из Python 2.x и 3.x
>>>> возвращать первым параметром "alt" для любых дистрибутивов с
>>>> /etc/altlinux-release.
>>>>
>>>> Сейчас оно берёт первое слово из /etc/altlinux-release, что приводит к
>>>> https://bugzilla.altlinux.org/show_bug.cgi?id=34266
>>>>
>>>> Возражения есть?
>>>>
>>> Твоё предложение выглядит не очень здраво и может кому-то что-то сломать.
>>>
>>> Вот тут есть ещё одна реализация для тех же целей:
>>>
>>> http://git.altlinux.org/tasks/205194/gears/400/git?p=git;a=commitdiff;h=330fa74425e6728803517bbd84348c0c2b7d921f
>> А что она реализует?
>>
>>
>>
>>
>> _______________________________________________
>> Devel mailing list
>> Devel@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/devel
>
>
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel

Re: [devel] linux_distribution() из Python

[Mikhail Efremov]

On Fri, 4 May 2018 14:15:04 +0300 Andrey Bychkov wrote:
> Использует файл идентификации /etc/os-release (если он имеется) для 
> полочения информации о дистрибутиве. Парсит поля файла, а именно
> NAME, VERSION_ID, ID. Если нет, то информация извлекается старым
> способом.

И такой патч вполне можно отправлять в апстрим, кстати.
Хотя у нас можно учитывать и наличие /etc/altlinux-release, если
нет /etc/os-release.

-- 
WBR, Mikhail Efremov

Re: [devel] linux_distribution() из Python

[Sergey V Turchin]

On Friday, 4 May 2018 14:31:32 MSK Mikhail Efremov wrote:
> On Fri, 4 May 2018 14:15:04 +0300 Andrey Bychkov wrote:
> > Использует файл идентификации /etc/os-release (если он имеется) для
> > полочения информации о дистрибутиве. Парсит поля файла, а именно
> > NAME, VERSION_ID, ID. Если нет, то информация извлекается старым
> > способом.
> 
> И такой патч вполне можно отправлять в апстрим, кстати.
> Хотя у нас можно учитывать и наличие /etc/altlinux-release, если
> нет /etc/os-release.
Так и происходит.

-- 
Regards, Sergey.

Re: [devel] linux_distribution() из Python

[Sergey V Turchin]

On Friday, 4 May 2018 14:30:52 MSK Anton Farygin wrote:
> https://www.freedesktop.org/software/systemd/man/os-release.html
> 
> $ cat /etc/os-release
> NAME="ALT"
> VERSION="8.2 "
> ID=altlinux
> VERSION_ID=8.2
> PRETTY_NAME="ALT Workstation K 8.2  (Centaurea Ruthenica)"
> ANSI_COLOR="1;33"
> CPE_NAME="cpe:/o:alt:kworkstation:8.2"
> HOME_URL="http://www.basealt.ru"
> BUG_REPORT_URL="https://bugs.altlinux.org/"
> 
> Если я правильно понимаю, что с версии 8.0 у нас этот файл всегда
> корректно заполнен.
Да.

-- 
Regards, Sergey.

Re: [devel] linux_distribution() из Python

[Mikhail Efremov]

On Fri, 04 May 2018 14:37:12 +0300 Sergey V Turchin wrote:
> On Friday, 4 May 2018 14:31:32 MSK Mikhail Efremov wrote:
> > On Fri, 4 May 2018 14:15:04 +0300 Andrey Bychkov wrote:  
> > > Использует файл идентификации /etc/os-release (если он имеется)
> > > для полочения информации о дистрибутиве. Парсит поля файла, а
> > > именно NAME, VERSION_ID, ID. Если нет, то информация извлекается
> > > старым способом.  
> > 
> > И такой патч вполне можно отправлять в апстрим, кстати.
> > Хотя у нас можно учитывать и наличие /etc/altlinux-release, если
> > нет /etc/os-release.  
> Так и происходит.

Тогда это тоже можно попробовать заапстримить, в принципе. Только
отдельным патчем, разумеется.

-- 
WBR, Mikhail Efremov

Re: [devel] linux_distribution() из Python

[Sergey V Turchin]

On Saturday, 5 May 2018 01:27:49 MSK Mikhail Efremov wrote:

[...]
> > > Хотя у нас можно учитывать и наличие /etc/altlinux-release, если
> > > нет /etc/os-release.
> > 
> > Так и происходит.
> 
> Тогда это тоже можно попробовать заапстримить, в принципе. Только
> отдельным патчем, разумеется.
Оно и есть отдельный патч.
Если есть /etc/os-release, используем его, иначе делаем, как раньше.
Если речь про именно /etc/altlinux-release, то не вижу смысла рождать 
анахронизм.

-- 
Regards, Sergey.