ALT Linux Team development discussions
 help / color / mirror / Atom feed
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


  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