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] Помогите разобраться с пакетированием
Date: Fri, 4 Jul 2014 22:21:22 +0400
Message-ID: <20140704182122.GA12144@altlinux.org> (raw)
In-Reply-To: <m3lhs9owfk.fsf@thinkpad.evg-krsk.dyndns.org>

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

On Sat, Jul 05, 2014 at 01:39:27AM +0800, Евгений Терешков wrote:
> "Dmitry V. Levin" пишет:
> 
> > Этот /var/lib/vz/vz образовался в результате неправильного использования ln(1).
> 
> Угу.
> 
> >> Я прочитал test(1) на предмет ключа "-a" и поправил спек так (4.7.2-alt2,
> >> 4d6158e018a87fb68d1ca846443fef0b8e28a48d):
> >> ===8<=====================================================================
> >> %post
> >> # Some use /vz instead of /var/lib/vz; create a compatibility symlink
> >> test -e /vz || ln -s /var/lib/vz /
> >> 
> >> %preun
> >> test -L /vz && rm -f /vz
> >> ===8<=====================================================================
> 
> Только сейчас заметил, что в оригинальном ovz-шном спеке vzctl тоже везде
> "test -a". Неужели у них какой-то свой test?

Нет, test (что builtin, что в coreutils) в отношении -a/-e, скорее всего,
везде одинаковый.

> >> Т.е. заменил всегда возвращающее ненулевой код "test -a /vz" на "test -e" и
> >> "test -L".
> >> Безуспешно: по обновлению линк /vz всё равно не появляется. Интересно, что
> >> он 100% появляется, если переустановить версию 4.7.2-alt2 с помощью apt-get
> >> --reinstall install vzctl
> > Пожалуйста, уберите эту самодеятельность с /vz из пакета vzctl!
> 
> Хорошо, но как сделать правильно?. В частности, я столкнулся с тем что этот
> путь зашит в vzmigrate. Могу его запатчить, но не факт, что ещё что-то
> внешнее по отношению к vzctl не закладывается на /vz

За те 4.5 года, пока я собирал пакет vzctl, /vz нигде зашит не был,
и никаких проблем с этим не было.  И сейчас тоже нигде не должно быть
кода, завязанного на существование /vz.  Если найдете, и если выяснится,
что это апстримный код - смело отправляйте патч Кириллу.

> >> Также себя ведёт и ещё один симлик из того же исходного коммита,
> >> /etc/net/ifup-post-local -> /usr/sbin/vzifup-post, т.е. они вместе с /vz не
> >> появляются при обновлении 4.5.1-alt1 => 4.7.2-alt1, но появляются при
> >> переустановке крайней версии.
> >> Помогите разобраться, что я делаю не так?
> > Это еще до вас, насколько я понимаю, неправильно написаны скрипты:
> > сперва %post делает ln, а потом %preun делает rm.
> 
> Да, подзабыл я уже порядок срабатывания триггеров. Спасибо.
> 
> P.S.: присоединяюсь к вопросу в соседнем письме относительно цели переноса
> /vz в /var/lib/vz.

Для того, чтобы пакет мог себе позволить создавать новые сущности
непосредственно в корневом каталоге, должна быть очевидная необходимость.
Если с 2006 года, когда vzctl был впервые упакован в Сизиф, я такой
необходимости не замечал, значит, она совсем не очевидная.

У /vz есть исторические корни - так было удобно разработчикам vz еще
до того, как появился openvz.  В Сизифе необходимости поддерживать
совместимость с установками, сделанными до openvz, никогда не было.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 181 bytes --]

  parent reply	other threads:[~2014-07-04 18:21 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-03 14:33 Евгений Терешков
2014-07-04 17:09 ` Dmitry V. Levin
2014-07-04 17:20   ` Michael Shigorin
2014-07-04 17:40     ` Евгений Терешков
2014-07-04 17:39   ` Евгений Терешков
2014-07-04 17:44     ` Led
2014-07-04 17:52       ` Евгений Терешков
2014-07-04 18:21     ` Dmitry V. Levin [this message]
2014-07-05 18:31     ` Денис Смирнов
2014-07-08 23:54     ` Денис Смирнов
2014-07-09  0:27       ` Евгений Терешков
2014-07-09  0:48         ` Денис Смирнов

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=20140704182122.GA12144@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