On 28-Jun-06, at 1:57 AM, Ask Bjørn Hansen wrote:
2) Make load_plugins or Qpsmtpd::Plugin smart enough that we
won't need plugin_name in each plugin.
That's a trivial change to your patch. Just make line 68 of the patch:
+ my $eval = require $package;\nsub ${plugin}::plugin_name
On Wed, Jun 28, 2006 at 10:41:51AM -0400, Matt Sergeant wrote:
+ my $eval = require $package;\nsub ${plugin}::plugin_name
{ '$plugin' };
Why the \n? It seems to me that a space would be more readable
and have the same effect.
Regards,
Andrew
--
mailto:[EMAIL PROTECTED]
On 28-Jun-06, at 11:43 AM, Andrew Pam wrote:
On Wed, Jun 28, 2006 at 10:41:51AM -0400, Matt Sergeant wrote:
+ my $eval = require $package;\nsub ${plugin}::plugin_name
{ '$plugin' };
Why the \n? It seems to me that a space would be more readable
and have the same effect.
Bikeshed
It's in the wiki now:
http://wiki.qpsmtpd.org/plugins#other_plugins
Still, would be nice if you could put it in the contrib/ dir. ;)
On Fri, Jun 23, 2006 at 09:52:46PM +0200, Robert Millan wrote:
Note that sendmail already supports it, but only in the recieving part. For
exim, I have a
Author: ask
Date: Wed Jun 28 13:05:04 2006
New Revision: 648
Modified:
branches/0.3x/ (props changed)
branches/0.3x/Changes
branches/0.3x/lib/Qpsmtpd.pm
branches/0.3x/lib/Qpsmtpd/Plugin.pm
Log:
[EMAIL PROTECTED]: ask | 2006-06-28 13:04:50 -0700
Support module plugins (My::Plugin
Author: msergeant
Date: Wed Jun 28 16:27:40 2006
New Revision: 649
Modified:
trunk/lib/Danga/DNS/Resolver.pm
Log:
More fields work
Modified: trunk/lib/Danga/DNS/Resolver.pm
==
--- trunk/lib/Danga/DNS/Resolver.pm