From: Igor Vlasenko <vlasenko@imath.kiev.ua> To: devel@lists.altlinux.org Cc: at@altlinux.org Subject: [devel] sisyphus_check: рубить с плеча не так уж безопасно. Date: Mon, 28 Mar 2011 11:35:38 +0300 Message-ID: <20110328083538.GA16530@dad.imath.kiev.ua> (raw) Уважаемые коллеги, Уважаемый Алексей, еще раз извиняюсь, что поднимаю эту тему. Когда я внезапно узнал из письма Алексея про AT> sisyphus_check, которые запрещает зависимость на xorg-devel сначала был в шоке, но потом немного отошел. Конечно, плохо, что такие вещи делались без обсуждения и предупреждения, но Алексей сделает из этого какие-то выводы, подумал я, плюс, эти изменения действительно дают небольшую оптимизацию. Так что человек нещепетильный пусть уступит человеку щепетильному. Раз Алексею это важно, пусть будет. К сожалению, кода я получил сообщеня о непересобирающихся пакетах и посмотрел в код sisyphus_check, шок у меня случился повторно. В sisyphus_check Алексей запретил все пакеты вида xorg-x11-.* . Из этих пакетов желаемую Алексеем оптимизацию даст только борьба с xorg-x11-devel. Остальные (B)Req: пострадали безвинно, их "вина" только в том, что они не угнались за переименованиями в Сизифе. "Лечение" пострадавших пакетов заулючается в механических subst xorg-x11-server -> xorg-server xorg-x11-proto-devel -> xorg-proto-devel xorg-x11-locales -> libX11-locales и, как видим, совершенно бесполезно и идет вразрез с нашей практикой при переименовании пакета оставлять совместимые Provides:. Кроме того, Provides: вида xorg-x11-server, xorg-x11-proto-devel, нужны для совместимости с другими дистрибутивами, в частности, fedora, чтобы облегчать написание кроссдистрибутивных rpm и уменьшать дурную работу при адаптации пакета из другого дистрибутива. Поэтому проверке xorg-x11-.* не место в sisyphus_check, из уважения к Алексею ее можно заменить на xorg-x11-devel. Вот списки безвинно пострадавших (B)Req: sqlite> select distinct requirename from rpm_requires where requirename glob 'xorg-x11-*'; xorg-x11-drv-vboxmouse xorg-x11-drv-vboxvideo xorg-x11-drv-video xorg-x11-font-utils xorg-x11-libs xorg-x11-locales xorg-x11-proto-devel xorg-x11-rgb xorg-x11-server xorg-x11-utils xorg-x11-xauth xorg-x11-xnest xorg-x11-xvfb sqlite> select distinct buildrequirename from rpm_buildrequires where buildrequirename glob 'xorg-x11-*'; xorg-x11-bitmaps xorg-x11-font-utils xorg-x11-libs xorg-x11-locales xorg-x11-mesagl xorg-x11-proto-devel xorg-x11-server xorg-x11-util-macros xorg-x11-utils xorg-x11-xtrans-devel xorg-x11-xvfb -- Dr. Igor Vlasenko -------------------- Topology Department Institute of Math Kiev, Ukraine
next reply other threads:[~2011-03-28 8:35 UTC|newest] Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-03-28 8:35 Igor Vlasenko [this message] 2011-03-28 8:49 ` REAL 2011-03-28 9:32 ` Dmitry V. Levin 2011-03-28 9:36 ` [devel] sisyphus_check, "you do it" и взаимоуважение Michael Shigorin 2011-03-28 9:46 ` Aleksey Novodvorsky 2011-03-28 9:58 ` Yuri N. Sedunov 2011-03-28 9:59 ` Michael Shigorin
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=20110328083538.GA16530@dad.imath.kiev.ua \ --to=vlasenko@imath.kiev.ua \ --cc=at@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