* [devel] %{get_dep somelib} deprecation
@ 2011-01-29 21:48 Igor Vlasenko
2011-01-29 23:06 ` Alexey Tourbin
0 siblings, 1 reply; 7+ messages in thread
From: Igor Vlasenko @ 2011-01-29 21:48 UTC (permalink / raw)
To: devel
Уважаемые коллеги,
извиняюсь за опоздание, добавил к 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] %{get_dep somelib} deprecation
2011-01-29 21:48 [devel] %{get_dep somelib} deprecation Igor Vlasenko
@ 2011-01-29 23:06 ` Alexey Tourbin
2011-01-30 18:19 ` Igor Vlasenko
0 siblings, 1 reply; 7+ messages in thread
From: Alexey Tourbin @ 2011-01-29 23:06 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] %{get_dep somelib} deprecation
2011-01-29 23:06 ` Alexey Tourbin
@ 2011-01-30 18:19 ` Igor Vlasenko
2011-01-30 18:29 ` Alexey Tourbin
0 siblings, 1 reply; 7+ messages in thread
From: Igor Vlasenko @ 2011-01-30 18:19 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] %{get_dep somelib} deprecation
2011-01-30 18:19 ` Igor Vlasenko
@ 2011-01-30 18:29 ` Alexey Tourbin
2011-01-30 21:07 ` Igor Vlasenko
0 siblings, 1 reply; 7+ messages in thread
From: Alexey Tourbin @ 2011-01-30 18:29 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] %{get_dep somelib} deprecation
2011-01-30 18:29 ` Alexey Tourbin
@ 2011-01-30 21:07 ` Igor Vlasenko
2011-01-30 21:18 ` Alexey Tourbin
0 siblings, 1 reply; 7+ messages in thread
From: Igor Vlasenko @ 2011-01-30 21:07 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] %{get_dep somelib} deprecation
2011-01-30 21:07 ` Igor Vlasenko
@ 2011-01-30 21:18 ` Alexey Tourbin
2011-01-31 12:45 ` Sergey V Turchin
0 siblings, 1 reply; 7+ messages in thread
From: Alexey Tourbin @ 2011-01-30 21:18 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [devel] %{get_dep somelib} deprecation
2011-01-30 21:18 ` Alexey Tourbin
@ 2011-01-31 12:45 ` Sergey V Turchin
0 siblings, 0 replies; 7+ messages in thread
From: Sergey V Turchin @ 2011-01-31 12:45 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- 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 --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-01-31 12:45 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-29 21:48 [devel] %{get_dep somelib} deprecation Igor Vlasenko
2011-01-29 23:06 ` Alexey Tourbin
2011-01-30 18:19 ` Igor Vlasenko
2011-01-30 18:29 ` Alexey Tourbin
2011-01-30 21:07 ` Igor Vlasenko
2011-01-30 21:18 ` Alexey Tourbin
2011-01-31 12:45 ` Sergey V Turchin
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git