From: Alexey Tourbin <at@altlinux.ru> To: ALT Linux Team development discussions <devel@lists.altlinux.org> Cc: viy@altlinux.org Subject: Re: [devel] I: repocop test for %{get_version ...} is disabled. Date: Thu, 23 Aug 2012 00:53:34 +0400 Message-ID: <20120822205334.GA32338@altlinux.org> (raw) In-Reply-To: <20120822195649.GA23099@dad.imath.kiev.ua> On Wed, Aug 22, 2012 at 10:56:49PM +0300, Igor Vlasenko wrote: > Уважаемые коллеги, > по поводу макроса get_version в spec-файлах. > В свое время Алексей реализовал set:versions, > и тогда в devel@ были заявения, что теперь с set:versions %get_version > не нужен и его пора удалять, > в результате чего я даже написал тест репокоп, который удаляет в spec-файлах > макрос get_version, как, например, в > http://repocop.altlinux.org/pub/repocop/reports/diff/by-acl/zerg/poppler26-0.20.2-alt1.diff -Requires: libqt4-core >= %{get_version libqt4-core} +Requires: libqt4-core Надо было удалять всю зависимость на libqt4-core, а не только %get_version. > Пыль осела, и начал сомневаться, а нужно ли удалять %get_version? > В большинстве случаев он избыточен, но ведь вреда вроде бы от него нет? > > Я обсудил ситуацию с Сергеем (zerg@, на него робот приготовил большое NMU > с удалением %get_version в kde пакетах), и пришли к выводу, > что раз формально set:versions не дает полных гарантий, > то и убирать %get_version не надо, хуже от него не будет. Set-версии формально не дают полной гарантии, а какую тогда гарантию формально дает %get_version? Можно подумать, что указание версии, не меньшей текущей, дает полную гарантию, и разводить демагогию, а на самом деле гарантии нет вообще никакой. Нет никакой гарантии, что в следующих версиях сохранится полная обратная совместимость. А set-версии как раз пытаются выразить, и притом довольно успешно и совершенно автоматически, определенный критерий совместимости между версиями библиотек. То, что "хуже не будет", aka "от балды" - я тогда сейчас сделаю в rpmbuild, чтобы он и все зависимости дополнял текущими версиями, хуже не будет. А будет круто! В чем вообще смысл зависимости Requires: libqt4-core >= %{get_version libqt4-core} в системе с хорошо развитыми автоматическими зависимостями? Это какое-то ретроградство и оксюморон из 1998 года. Вообще, мега-идея была, чтобы не указывать зависимости вручную вообще: все Requires зависимости из спекфайлов снести, а BuildRequires зависимости добавлять только с помощью /usr/bin/buildreq. Понятно, что не предлагалось следовать этой идее с особым фанатизмом, но в то же время было многое сделано для того, чтобы в большинстве случаев получался приемлемый результат. Так что роботу следовало бы попросту косить все зависимости из спекфайлов под корень. :-) > В общем, отключил тест. Если есть возражения, прошу высказать. > > -- > > Dr. Igor Vlasenko > -------------------- > Topology Department > Institute of Math > Kiev, Ukraine
next prev parent reply other threads:[~2012-08-22 20:53 UTC|newest] Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-08-22 19:56 Igor Vlasenko 2012-08-22 20:53 ` Alexey Tourbin [this message] 2012-08-22 21:16 ` Michael Shigorin 2012-08-22 21:29 ` Dmitry V. Levin 2012-08-23 6:09 ` Igor Vlasenko 2012-08-23 7:42 ` Alexey Tourbin 2012-08-23 8:22 ` Igor Vlasenko 2012-08-23 10:33 ` Sergey V Turchin 2012-08-23 13:03 ` Alexey Tourbin 2012-08-23 15:26 ` Sergey V Turchin 2012-08-24 0:40 ` Alexey Tourbin 2012-08-24 10:47 ` Sergey V Turchin 2012-08-22 21:11 ` Денис Смирнов 2012-08-22 21:29 ` Alexey Tourbin 2012-08-22 23:24 ` Alexey Tourbin 2012-08-23 5:03 ` Денис Смирнов 2012-08-23 6:06 ` Alexey Tourbin 2012-08-23 8:04 ` Michael Shigorin 2012-08-23 8:58 ` Alexey Tourbin 2012-08-23 10:48 ` Michael Shigorin 2012-08-23 16:38 ` Денис Смирнов 2012-08-24 0:52 ` Alexey Tourbin 2012-08-24 4:12 ` Денис Смирнов 2012-08-24 14:13 ` Alexey Tourbin 2012-08-24 17:52 ` Денис Смирнов 2012-08-27 11:02 ` Ildar Mulyukov 2012-08-27 16:05 ` Alexey Tourbin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20120822205334.GA32338@altlinux.org \ --to=at@altlinux.ru \ --cc=devel@lists.altlinux.org \ --cc=viy@altlinux.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
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