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 --]
next prev 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