[devel] %{get_dep somelib} deprecation

[devel] %{get_dep somelib} deprecation

[Igor Vlasenko]

Уважаемые коллеги,
извиняюсь за опоздание, добавил к repocop тест
на использование макроса %{get_dep ...}

Саейчас он используется для следующих библиотек:
Requires: %{get_dep fuse}
Requires: %{get_dep kde4libs}
Requires: %{get_dep kdelibs}
Requires: %{get_dep kdelibs-devel}
Requires: %{get_dep libakonadiprotocolinternals}
Requires: %{get_dep libapt}
Requires: %{get_dep libgoogle-gadgets-qt}
Requires: %{get_dep libqt3}
Requires: %{get_dep libqt4-core}
Requires: %{get_dep libqzion}
Requires: %{get_dep libtag}
Requires: %{get_dep libtapioca-qt}
Requires: %{get_dep libwildmidi}
Requires: %{get_dep perl-base}
Requires: %{get_dep rep-gtk}
Requires: %{get_dep rpm}

Сейчас он вроде бы нигде не нужен, поэтому тест ругает
все случаи использования %{get_dep. Если же он таки где-то будет
нужен, сообщите пожалуйста, я буду добавлять в тест исключения.

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

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] %{get_dep somelib} deprecation

[Alexey Tourbin]

On Sat, Jan 29, 2011 at 11:48:47PM +0200, Igor Vlasenko wrote:
> Уважаемые коллеги,
> извиняюсь за опоздание, добавил к repocop тест
> на использование макроса %{get_dep ...}

Я собирался убрать из get_dep Release, то есть вместо зависимостей
Requires: N >= [E:]V-R
сделать 
Requires: N >= [E:]V

Rationale: релизы в сизифе и в бранчах устроены по-разному,
а совместимость, как правило, определяется апстримной версией.

Я согласен, что этот макрос лушче не использовать совсем.

> Саейчас он используется для следующих библиотек:
> Requires: %{get_dep fuse}
> Requires: %{get_dep kde4libs}
> Requires: %{get_dep kdelibs}
> Requires: %{get_dep kdelibs-devel}
> Requires: %{get_dep libakonadiprotocolinternals}
> Requires: %{get_dep libapt}
> Requires: %{get_dep libgoogle-gadgets-qt}
> Requires: %{get_dep libqt3}
> Requires: %{get_dep libqt4-core}
> Requires: %{get_dep libqzion}
> Requires: %{get_dep libtag}
> Requires: %{get_dep libtapioca-qt}
> Requires: %{get_dep libwildmidi}
> Requires: %{get_dep perl-base}
> Requires: %{get_dep rep-gtk}
> Requires: %{get_dep rpm}
> 
> Сейчас он вроде бы нигде не нужен, поэтому тест ругает
> все случаи использования %{get_dep. Если же он таки где-то будет
> нужен, сообщите пожалуйста, я буду добавлять в тест исключения.
> 
> Также, добавлен патчгенератор,
> если нужно, могу сделать NMU.
> 
> -- 
> 
> Dr. Igor Vlasenko
> --------------------
> Topology Department
> Institute of Math
> Kiev, Ukraine

Re: [devel] %{get_dep somelib} deprecation

[Igor Vlasenko]

On Sun, Jan 30, 2011 at 02:06:55AM +0300, Alexey Tourbin wrote:
> On Sat, Jan 29, 2011 at 11:48:47PM +0200, Igor Vlasenko wrote:
> > Уважаемые коллеги,
> > извиняюсь за опоздание, добавил к repocop тест
> > на использование макроса %{get_dep ...}
> 
> Я собирался убрать из get_dep Release, то есть вместо зависимостей
> Requires: N >= [E:]V-R
> сделать 
> Requires: N >= [E:]V
> 
> Rationale: релизы в сизифе и в бранчах устроены по-разному,
> а совместимость, как правило, определяется апстримной версией.

Правильная мысль. 

Кстати, я не собирался бороться с этим макросом,
просто все его текущие use cases выглядят как устаревшие.

Т.е. решил, что проще будет добавлять исключения в тест,
когда в %{get_dep будет необходимость.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] %{get_dep somelib} deprecation

[Alexey Tourbin]

On Sun, Jan 30, 2011 at 08:19:11PM +0200, Igor Vlasenko wrote:
> On Sun, Jan 30, 2011 at 02:06:55AM +0300, Alexey Tourbin wrote:
> > Я собирался убрать из get_dep Release, то есть вместо зависимостей
> > Requires: N >= [E:]V-R
> > сделать 
> > Requires: N >= [E:]V
> > 
> > Rationale: релизы в сизифе и в бранчах устроены по-разному,
> > а совместимость, как правило, определяется апстримной версией.
> 
> Правильная мысль. 
> 
> Кстати, я не собирался бороться с этим макросом,
> просто все его текущие use cases выглядят как устаревшие.
> 
> Т.е. решил, что проще будет добавлять исключения в тест,
> когда в %{get_dep будет необходимость.

В некоторых местах вместо get_dep используется get_version
или get_SVR.  Наверное, их нужно диагностировать точно так же,
как get_dep.

> -- 
> 
> Dr. Igor Vlasenko
> --------------------
> Topology Department
> Institute of Math
> Kiev, Ukraine
> 
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel

Re: [devel] %{get_dep somelib} deprecation

[Igor Vlasenko]

On Sun, Jan 30, 2011 at 09:29:59PM +0300, Alexey Tourbin wrote:
> В некоторых местах вместо get_dep используется get_version
> или get_SVR.  Наверное, их нужно диагностировать точно так же,
> как get_dep.

Спасибо, посмотрел, get_version испоьзовался по разному,
буду ругать use case вида
Requires: foo >= %{get_version bar}


-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine

Re: [devel] %{get_dep somelib} deprecation

[Alexey Tourbin]

On Sun, Jan 30, 2011 at 11:07:11PM +0200, Igor Vlasenko wrote:
> On Sun, Jan 30, 2011 at 09:29:59PM +0300, Alexey Tourbin wrote:
> > В некоторых местах вместо get_dep используется get_version
> > или get_SVR.  Наверное, их нужно диагностировать точно так же,
> > как get_dep.
> 
> Спасибо, посмотрел, get_version испоьзовался по разному,
> буду ругать use case вида
> Requires: foo >= %{get_version bar}

Это лучше, чем привязываться к релизу, но смысл почти такой же, как
у get_dep: привязка к _текущей_ версии пакета.  Зачем привязываться
к _текущей_ версии пакета?  Если это делается для обеспечения
совместимости, то...

Раньше это было в какой-то степени оправдано в Си+плюс пакетах,
типа КДЕ-шных, потому что у Си+плюса очень хрупкий ABI (изменение
типа аргумента - напр., добавление const - приводит к изменению имени
функции), и даже при изменении версии в последней цифре ABI как бы
мог немножко измениться.

Короче, эти грубые зависимости больше не нужны.

> -- 
> 
> Dr. Igor Vlasenko
> --------------------
> Topology Department
> Institute of Math
> Kiev, Ukraine

Re: [devel] %{get_dep somelib} deprecation

[Sergey V Turchin]

[-- Attachment #1: Type: Text/Plain, Size: 315 bytes --]

On Monday 31 January 2011 00:18, Alexey Tourbin wrote:

[...]
> Раньше это было в какой-то степени оправдано в Си+плюс пакетах,
> типа КДЕ-шных, потому что у Си+плюса очень хрупкий ABI

[...]
> Короче, эти грубые зависимости больше не нужны.
Согласен.

-- 
Regards, Sergey.       ALT Linux, http://www.altlinux.ru/

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]