From: Igor Vlasenko <vlasenko@imath.kiev.ua> To: ALT Linux Team development discussions <devel@lists.altlinux.org> Subject: Re: [devel] [#56408] DONE (try 52) perl.git=5.14.2-alt1 srpm=perl-Filter-1.39-alt1.src.rpm ... Date: Wed, 26 Oct 2011 21:57:05 +0300 Message-ID: <20111026185705.GA23459@dad.imath.kiev.ua> (raw) In-Reply-To: <20111026181420.GA8965@mail.truecrux.org> On Wed, Oct 26, 2011 at 10:14:20PM +0400, thecrux@gmail.com wrote: > Система хуков в таком виде смахивает на костыли и непонятно как её, > например, опакетить. Хорошее замечание. Например, хуки для репокопа я пакетил в /usr/share/repocop/fixscripts. Для общих хуков можно выделить что-то вроде /usr/share/srpmutils/hooks > Гораздо логичнее выглядит система плагинов, расширяющая функционал > базового модуля. Подключая нужный плагин, получаешь нужный результат. > Тесты для таких плагинов писать будет значительно проще и паковать в > виде обычных perl-пакетов. Хук, вообще говоря, и есть простейший плагин, загружаемый через requires; Если попытаться по-другому, в итоге выходит слишком много букф. Ведь плагин надо как-то инициализировать, и вызвать. у хука на это уходит 3 строчки, 55 символов. ----------------------- push @SPECHOOKS, sub { my ($spec,$parent) = @_; }; ----------------------- Остальное уже полезный код. С другой стороны, есть и тяжелый код, который логично размещать в отдельных модулях. Хочу для такого зарезервировать namespace RPM::Source::Tools::* Пока там только RPM::Source::Tools::Uupdate.pm -- Dr. Igor Vlasenko -------------------- Topology Department Institute of Math Kiev, Ukraine -- This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean.
next prev parent reply other threads:[~2011-10-26 18:57 UTC|newest] Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-10-24 22:36 ` Alexey Tourbin 2011-10-24 23:18 ` Dmitry V. Levin 2011-10-29 23:40 ` [devel] perl.h -D_FILE_OFFSET_BITS=64 Dmitry V. Levin 2011-11-01 23:28 ` Dmitry V. Levin 2011-10-25 7:13 ` [devel] [#56408] DONE (try 52) perl.git=5.14.2-alt1 srpm=perl-Filter-1.39-alt1.src.rpm thecrux 2011-10-25 13:51 ` Michael Shigorin 2011-10-26 14:50 ` Igor Vlasenko 2011-10-26 16:56 ` Igor Vlasenko 2011-10-26 18:14 ` thecrux 2011-10-26 18:57 ` Igor Vlasenko [this message] 2011-10-26 19:14 ` Igor Vlasenko
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=20111026185705.GA23459@dad.imath.kiev.ua \ --to=vlasenko@imath.kiev.ua \ --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