ALT Linux Team development discussions
 help / color / mirror / Atom feed
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.



  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