Status: CLOSED NOTABUG
Alias: None
Product: Sisyphus
Classification: Development
Component: spirv-tools (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 enhancement
Assignee: Денис Назаров
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-09 15:50 MSK by Sergey V Turchin
Modified: 2021-02-10 16:23 MSK (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey V Turchin 2021-02-09 15:50:20 MSK
Почему, собственно, в soname именно 19? Я не нашёл ничего подобного. Откуда оно?

Хочется выяснить и исправить в случае ошибки до попадания в стабильный бранч.
Comment 1 Sergey V Turchin 2021-02-09 15:55:16 MSK
По glslang тот же вопрос.
Comment 2 Денис Назаров 2021-02-09 20:11:28 MSK
Т.к. ABI фактически сменился что у spirv-tools, что у glslang, то согласно https://www.altlinux.org/Shared_Libs_Policy следует сменить номер в названии подпакета. Т.к. разработчики не используют нормальную нумерацию soname'ов, то (что бы не придумывать самому) решил взять нумеразию из openSUSE - https://build.opensuse.org/package/show/openSUSE%3AFactory/spirv-tools и https://build.opensuse.org/package/show/openSUSE%3AFactory/glslang
Comment 3 Sergey V Turchin 2021-02-10 11:03:08 MSK
(Ответ для Назаров Денис на комментарий #2)
> Т.к. ABI фактически сменился что у spirv-tools, что у glslang, то согласно
> https://www.altlinux.org/Shared_Libs_Policy следует сменить номер в названии
> подпакета.
У нас это уже не обязательно т.к. уже имеем set-versions.

> то (что бы не придумывать самому) решил взять нумеразию из openSUSE -
Дык, у них там вообще "suse19".
Я лично не против смены soname, но предлагаю не делать своё, а в точности, как хоть у кого-нибудь или оставить, как было.
Comment 4 Денис Назаров 2021-02-10 11:07:35 MSK
(Ответ для Sergey V Turchin на комментарий #3)
> У нас это уже не обязательно т.к. уже имеем set-versions.

Про это не знал (
Но если оставить 0, то как оставить библиотеку со старым ABI (что бы всё не пересобирать) и новым?
 
> Дык, у них там вообще "suse19".
> Я лично не против смены soname, но предлагаю не делать своё, а в точности,
> как хоть у кого-нибудь или оставить, как было.

Ну не suse19 же писать )
А alt19 смахивает на тег, поэтому решил сделать просто 19.
Comment 5 Sergey V Turchin 2021-02-10 11:23:44 MSK
(Ответ для Назаров Денис на комментарий #4)
> > У нас это уже не обязательно т.к. уже имеем set-versions.
> Про это не знал (
Не. Оно вообще правильно сменить soname. Просто, смотреть надо на величину последствий. Конкретно тут, думаю, лучше сменить, чтоб при обновлении с бранча на бранч полсистемы не вынесло.

> Но если оставить 0, то как оставить библиотеку со старым ABI (что бы всё не
> пересобирать) и новым?
Пересобрать, если сборочница скажет, что ABI нарушен.

> Ну не suse19 же писать )
> А alt19 смахивает на тег, поэтому решил сделать просто 19.
Я бы проверил, что у всех возможных. У Fedora/Debian/Ubuntu/Gentoo не трогают ничего, т.е. не как у нас в любом случае. Возможно, текущий вариант самый правильный.
Comment 6 Sergey V Turchin 2021-02-10 12:16:14 MSK
> Konstantin A Lepikhov (L.A. Kostis) <lakostis@alt> changed:
> * CC: lakostis@alt =>
И от рассылок отписался? ;-)
Comment 7 Sergey V Turchin 2021-02-10 16:23:05 MSK
> Возможно, текущий вариант самый правильный.