[devel] libdevmapper and versioning

[devel] libdevmapper and versioning

[Alexey Shabalin]

Вечер добрый.
Я подозреваю что к libdevmapper(из пакета lvm2) не правильно приделано
версионирование.
Смущает меня указание:
local:
  *;
во многих интерфейсах.
Прошу поддтвердить/опровергнуть более сведующих людей.
И может быть at@ пересоберёт lvm2 с оторванным версионированием.

-- 
Alexey Shabalin

Re: [devel] libdevmapper and versioning

[Dmitry V. Levin]

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

On Tue, Feb 15, 2011 at 12:09:18AM +0300, Alexey Shabalin wrote:
> Вечер добрый.
> Я подозреваю что к libdevmapper(из пакета lvm2) не правильно приделано
> версионирование.
> Смущает меня указание:
> local:
>   *;
> во многих интерфейсах.
> Прошу поддтвердить/опровергнуть более сведующих людей.

Мне тоже кажется, что версионирование там какое-то спорадическое.
Это, впрочем, не значит, что ограничение множества экспортируемых
символов там не требуется.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] libdevmapper and versioning

[Alexey Shabalin]

2011/2/15 Dmitry V. Levin:
> On Tue, Feb 15, 2011 at 12:09:18AM +0300, Alexey Shabalin wrote:
>> Вечер добрый.
>> Я подозреваю что к libdevmapper(из пакета lvm2) не правильно приделано
>> версионирование.
>> Смущает меня указание:
>> local:
>>   *;
>> во многих интерфейсах.
>> Прошу поддтвердить/опровергнуть более сведующих людей.
>
> Мне тоже кажется, что версионирование там какое-то спорадическое.
> Это, впрочем, не значит, что ограничение множества экспортируемых
> символов там не требуется.

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

-- 
Alexey Shabalin

Re: [devel] libdevmapper and versioning

[Dmitry V. Levin]

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

On Tue, Feb 15, 2011 at 02:07:53PM +0300, Alexey Shabalin wrote:
> 2011/2/15 Dmitry V. Levin:
> > On Tue, Feb 15, 2011 at 12:09:18AM +0300, Alexey Shabalin wrote:
> >> Вечер добрый.
> >> Я подозреваю что к libdevmapper(из пакета lvm2) не правильно приделано
> >> версионирование.
> >> Смущает меня указание:
> >> local:
> >>   *;
> >> во многих интерфейсах.
> >> Прошу поддтвердить/опровергнуть более сведующих людей.
> >
> > Мне тоже кажется, что версионирование там какое-то спорадическое.
> > Это, впрочем, не значит, что ограничение множества экспортируемых
> > символов там не требуется.
> 
> может тогда правильнее увеличить сонейм

Нет, soname менять вроде бы совершенно незачем.

> и сделать только ограничение экспортируемых символов,

Разве что скриптом, иначе сложно будет поддерживать.

> без всякого версионирования?

Какие-то остатки версионирования нужны для обратной совместимости.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

Re: [devel] libdevmapper and versioning

[Alexey Shabalin]

2011/2/15 Dmitry V. Levin <ldv@altlinux.org>:
> On Tue, Feb 15, 2011 at 02:07:53PM +0300, Alexey Shabalin wrote:
>> 2011/2/15 Dmitry V. Levin:
>> > On Tue, Feb 15, 2011 at 12:09:18AM +0300, Alexey Shabalin wrote:
>> >> Вечер добрый.
>> >> Я подозреваю что к libdevmapper(из пакета lvm2) не правильно приделано
>> >> версионирование.
>> >> Смущает меня указание:
>> >> local:
>> >>   *;
>> >> во многих интерфейсах.
>> >> Прошу поддтвердить/опровергнуть более сведующих людей.
>> >
>> > Мне тоже кажется, что версионирование там какое-то спорадическое.
>> > Это, впрочем, не значит, что ограничение множества экспортируемых
>> > символов там не требуется.
>>
>> может тогда правильнее увеличить сонейм
>
> Нет, soname менять вроде бы совершенно незачем.
>
ок.

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

>> без всякого версионирования?
>
> Какие-то остатки версионирования нужны для обратной совместимости.

Кроме этого смущает патч lvm2-2.02.82-alt-ldflags.patch:
-LDFLAGS="$LDFLAGS -Wl,--export-dynamic"
+LDFLAGS="$LDFLAGS"
для статики это конечно помешает, а вот для shared? с учётом что
dlopen используется в dmeventd

-- 
Alexey Shabalin

Re: [devel] libdevmapper and versioning

[Alexey Shabalin]

2011/2/15 Alexey Shabalin:
> 2011/2/15 Dmitry V. Levin :
>>> >> Вечер добрый.
>>> >> Я подозреваю что к libdevmapper(из пакета lvm2) не правильно приделано
>>> >> версионирование.
>>> >> Смущает меня указание:
>>> >> local:
>>> >>   *;
>>> >> во многих интерфейсах.
>>> >> Прошу поддтвердить/опровергнуть более сведующих людей.
>>> >
>>> > Мне тоже кажется, что версионирование там какое-то спорадическое.
>>> > Это, впрочем, не значит, что ограничение множества экспортируемых
>>> > символов там не требуется.
>>>
>>> и сделать только ограничение экспортируемых символов,
>> Разве что скриптом, иначе сложно будет поддерживать.
> там апстрим это и так делает

>> Какие-то остатки версионирования нужны для обратной совместимости.
>
> Кроме этого смущает патч lvm2-2.02.82-alt-ldflags.patch:
> -LDFLAGS="$LDFLAGS -Wl,--export-dynamic"
> +LDFLAGS="$LDFLAGS"
> для статики это конечно помешает, а вот для shared? с учётом что
> dlopen используется в dmeventd
В качестве замены альтового патча lvm2-2.02.82-alt-ldflags.patch лучше
взять патч от debian libs-cleanup.patch
Посмотреть можно у меня в git.
Правильный уход от версионирования не делал - не знаю как.

-- 
Alexey Shabalin

Re: [devel] libdevmapper and versioning

[Vitaly Kuznetsov]

Alexey Shabalin <a.shabalin@gmail.com> writes:

>>> Какие-то остатки версионирования нужны для обратной совместимости.
>>
>> Кроме этого смущает патч lvm2-2.02.82-alt-ldflags.patch:
>> -LDFLAGS="$LDFLAGS -Wl,--export-dynamic"
>> +LDFLAGS="$LDFLAGS"
>> для статики это конечно помешает, а вот для shared? с учётом что
>> dlopen используется в dmeventd
> В качестве замены альтового патча lvm2-2.02.82-alt-ldflags.patch лучше
> взять патч от debian libs-cleanup.patch
> Посмотреть можно у меня в git.
> Правильный уход от версионирования не делал - не знаю как.

Вобщем, по результатам #38689 наше версионирование проще оторвать без
остатков, пересборки потребует 6 пакетов. Просьба мейнтейнерам,
получившим соответствующее письмо, присоединиться к #38689.
В частности, хотелось бы там увидеть майнтейнеров libvirt, parted и
cryptsetup. Либо acl.

-- 
Vitaly Kuznetsov, ALT Linux

Re: [devel] libdevmapper and versioning

[Alexey Tourbin]

On Tue, Feb 15, 2011 at 05:41:57PM +0000, Vitaly Kuznetsov wrote:
> Alexey Shabalin <a.shabalin@gmail.com> writes:
> 
> >>> Какие-то остатки версионирования нужны для обратной совместимости.
> >>
> >> Кроме этого смущает патч lvm2-2.02.82-alt-ldflags.patch:
> >> -LDFLAGS="$LDFLAGS -Wl,--export-dynamic"
> >> +LDFLAGS="$LDFLAGS"
> >> для статики это конечно помешает, а вот для shared? с учётом что
> >> dlopen используется в dmeventd
> > В качестве замены альтового патча lvm2-2.02.82-alt-ldflags.patch лучше
> > взять патч от debian libs-cleanup.patch
> > Посмотреть можно у меня в git.
> > Правильный уход от версионирования не делал - не знаю как.
> 
> Вобщем, по результатам #38689 наше версионирование проще оторвать без
> остатков, пересборки потребует 6 пакетов. Просьба мейнтейнерам,

Это может быть опасно при обновлении, т.к. библиотека без версионирования
встаёт раньше пакетов с версионированными ссылками, которые будут
обновлены или удалены.  При утере обратной совместимости топологическая
сотртировка оборачивается медвежьей услугой!

Впрочем, скорее всего, это не очень опасно.

> получившим соответствующее письмо, присоединиться к #38689.
> В частности, хотелось бы там увидеть майнтейнеров libvirt, parted и
> cryptsetup. Либо acl.
> 
> -- 
> Vitaly Kuznetsov, ALT Linux

Re: [devel] libdevmapper and versioning

[Alexey Shabalin]

2011/2/15 Vitaly Kuznetsov :
> Alexey Shabalin <a.shabalin@gmail.com> writes:
>
>>>> Какие-то остатки версионирования нужны для обратной совместимости.
>>>
>>> Кроме этого смущает патч lvm2-2.02.82-alt-ldflags.patch:
>>> -LDFLAGS="$LDFLAGS -Wl,--export-dynamic"
>>> +LDFLAGS="$LDFLAGS"
>>> для статики это конечно помешает, а вот для shared? с учётом что
>>> dlopen используется в dmeventd
>> В качестве замены альтового патча lvm2-2.02.82-alt-ldflags.patch лучше
>> взять патч от debian libs-cleanup.patch
>> Посмотреть можно у меня в git.
>> Правильный уход от версионирования не делал - не знаю как.
>
> Вобщем, по результатам #38689 наше версионирование проще оторвать без
> остатков, пересборки потребует 6 пакетов. Просьба мейнтейнерам,
> получившим соответствующее письмо, присоединиться к #38689.
> В частности, хотелось бы там увидеть майнтейнеров libvirt, parted и
> cryptsetup. Либо acl.
libvirt добавил.
по поводу cryptsetup, в acl только naf@. он ещё что-нибудь собирает в
сизиф? а то у меня готова новая версия cryptsetup :)

-- 
Alexey Shabalin

Re: [devel] libdevmapper and versioning

[Vitaly Kuznetsov]

Alexey Shabalin <a.shabalin@gmail.com> writes:

> 2011/2/15 Vitaly Kuznetsov :
>> Alexey Shabalin <a.shabalin@gmail.com> writes:
>>
>>>>> Какие-то остатки версионирования нужны для обратной
> совместимости.
>>>>
>>>> Кроме этого смущает патч lvm2-2.02.82-alt-ldflags.patch:
>>>> -LDFLAGS="$LDFLAGS -Wl,--export-dynamic"
>>>> +LDFLAGS="$LDFLAGS"
>>>> для статики это конечно помешает, а вот для shared? с учётом что
>>>> dlopen используется в dmeventd
>>> В качестве замены альтового патча lvm2-2.02.82-alt-ldflags.patch
> лучше
>>> взять патч от debian libs-cleanup.patch
>>> Посмотреть можно у меня в git.
>>> Правильный уход от версионирования не делал - не знаю как.
>>
>> Вобщем, по результатам #38689 наше версионирование проще оторвать
> без
>> остатков, пересборки потребует 6 пакетов. Просьба мейнтейнерам,
>> получившим соответствующее письмо, присоединиться к #38689.
>> В частности, хотелось бы там увидеть майнтейнеров libvirt, parted и
>> cryptsetup. Либо acl.
> libvirt добавил.
> по поводу cryptsetup, в acl только naf@. он ещё что-нибудь собирает в
> сизиф? а то у меня готова новая версия cryptsetup :)

cryptsetup у нас совсем древний, майнтейнеру он, скорее всего, не
интересен. Добавляйте туда же.

-- 
Vitaly Kuznetsov, ALT Linux

Re: [devel] libdevmapper and versioning

[Dmitry V. Levin]

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

On Tue, Feb 15, 2011 at 06:31:58PM +0300, Alexey Shabalin wrote:
> 2011/2/15 Alexey Shabalin:
> > Кроме этого смущает патч lvm2-2.02.82-alt-ldflags.patch:
> > -LDFLAGS="$LDFLAGS -Wl,--export-dynamic"
> > +LDFLAGS="$LDFLAGS"
> > для статики это конечно помешает, а вот для shared? с учётом что
> > dlopen используется в dmeventd
> В качестве замены альтового патча lvm2-2.02.82-alt-ldflags.patch лучше
> взять патч от debian libs-cleanup.patch

Этот патч тоже выкидывает -Wl,--export-dynamic.
Если нет уверенности в том, что все плагины, которые может загрузить
dmeventd, долинкованы, добавьте в спек-файл
%set_verify_elf_method strict


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]