ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] [bug 10382] %_localstatedir: /var или /var/lib ?
Date: Wed, 25 Oct 2017 15:59:14 +0300
Message-ID: <20171025125914.GE16208@altlinux.org> (raw)
In-Reply-To: <416491a1-11dc-2e02-f92a-917f10b00561@cg.ru>

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

On Wed, Oct 25, 2017 at 03:31:46PM +0300, Lenar Shakirov wrote:
[...]
> На мой взгляд вы выбрали бесконечный путь.

У нас Sisyphus, куда уж бесконечнее! :)

> Любой пакет, упакованный в ALT, потенциально подвержен bug 10382,
> ибо проверки в sisyphus_check и rpm-build - это полумера, вы это и сами 
> знаете:
> http://git.altlinux.org/people/ldv/packages/openldap.git?p=openldap.git;a=commitdiff;h=6865c946b6bf9a4f873de07e5f8d3b8850824a2b

Ну вот баги в openldap нынешний rpm-build выявил бы.

Кстати, вы заметили, как в этом ненаглядном openldap называется макрос,
который получает значение %_localstatedir?  Он называется LDAP_RUNDIR!
И старое определение макроса LDAPI_SOCK:
#define LDAPI_SOCK LDAP_RUNDIR LDAP_DIRSEP "run" LDAP_DIRSEP "ldapi"
- что это, если не костыль, который кто-то влепил, полагаясь на то,
что в LDAP_RUNDIR никогда не будет /run?

> Некорректный путь может быть зашит везде: в man-файлах, unit-файлах, в 
> бинарниках.

Конечно, но там его легко найти.  Можно, конечно, представить себе, что
путь составляется побуквенно в рантайме, но мы всё-таки в первую очередь
хотим найти и исправить реалистичные случаи.
> 
> С годами проблема будет только ухудшаться - количество пакетов в репо 
> растет.
> 
> Возможно я ошибаюсь, но исправить localstatedir и поддерживать 
> _какое-то_ время бранчи проще, чем _бесконечно_ тратить силы на 
> совместимость.

Если /var/lib/lib и /var/lib/run будут выявляться и забраковываться
автоматически, то стоимость поддержки будет стремиться к нулю.

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

Когда вы предлагаете такие изменения, они должны быть обоснованы
не только благими пожеланиями, но и достоверной оценкой
непосредственных последствий.

Вот вы можете предложить реалистичный способ точно также надёжно
и автоматически выявлять и забраковывать неправильные пути в файлах
при смене localstatedir?


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

  reply	other threads:[~2017-10-25 12:59 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-09 11:42 Sergey Afonin
2017-10-17 22:35 ` Vitaly Lipatov
2017-10-18 11:56   ` Sergey Afonin
2017-10-25  5:14     ` Dmitry V. Levin
2017-10-25  5:38       ` Anton Farygin
2017-10-25  5:52         ` Dmitry V. Levin
2017-10-25  6:27           ` Anton Farygin
2017-10-25  6:50             ` Dmitry V. Levin
2017-10-25  7:26               ` Anton Farygin
2017-10-25 12:31               ` Lenar Shakirov
2017-10-25 12:59                 ` Dmitry V. Levin [this message]
2017-10-25 13:08                   ` Anton Farygin
2017-10-25 13:16                     ` Dmitry V. Levin
2017-10-25 13:17                       ` Anton Farygin
2017-10-25 13:27                         ` Dmitry V. Levin
2017-10-26 14:38                           ` Dmitry V. Levin
2017-10-26 15:23                             ` Michael Shigorin
2017-10-28  7:44                             ` Sergey Y. Afonin
2017-10-28  8:20                               ` Sergey Y. Afonin
2017-10-25 13:17                   ` Aleksei Nikiforov
2017-10-25 13:20                     ` [devel] автоматическая забраковка по содержимому файлов Dmitry V. Levin
2017-10-25 13:24                       ` Aleksei Nikiforov
2017-10-25 13:29                         ` Dmitry V. Levin
2017-10-26 18:32                           ` Dmitry V. Levin

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=20171025125914.GE16208@altlinux.org \
    --to=ldv@altlinux.org \
    --cc=devel@lists.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