Fwd: Re: [ingo] Re: [dev] Re: [cvs] commit: ingo/lib Driver.php ingo/lib/Driver client.php null.php timsieved.php

[Michael M Slusarz]

Sorry Brent - I finally found time to respond to this...
FWIW, I think your "3-pronged" approach is the way to go.  I will try to
abstract out the preferences stuff soon (today?) so we can get that out of
the way...

michael

----- Forwarded message from bjn at horde.org -----
    Date: Fri, 11 Apr 2003 15:29:20 -0500 (CDT)
    From: "Brent J. Nordquist" <bjn at horde.org>
Reply-To: "Brent J. Nordquist" <bjn at horde.org>
 Subject: Re: [ingo] Re: [dev] Re: [cvs] commit: ingo/lib Driver.php
ingo/lib/Driver client.php null.php timsieved.php
      To: Michael M Slusarz <slusarz at bigworm.colorado.edu>

On Fri, 11 Apr 2003, Michael M Slusarz <slusarz at bigworm.colorado.edu> wrote:

> It looks like what I (or you) need to do is to create a separate driver
> system for 'Ingo_Script' that deals with the actual generation and/or
> performance of the filters.  'Ingo_Driver' makes sense as the item to
> 'move' the rules to where it needs to live.  So I will (eventually)
> remove 'client' from Ingo_Driver and instead change it to 'Ingo_Script'.

Ah, I was wondering about that.  OK.  Well, I'm going to start working on
the actual guts of the procmail code.  If you have time/interest to
refactor the existing code to Ingo_Script with pluggable implementations,
great; I'll work my stuff into it.  If it's not there yet when I come to
need it I'll try my hand at it.

> BTW, I was thinking that it might be a better idea (or an additional
> option) to store the filter/blacklist/whitelist information in SQL
> instead of prefs.

Absolutely; I've thought the same myself (though it's my lowest priority
since prefs will work for us initially).  So now we actually have three
drivers :-)

 - rule storage: Ingo_Storage (OK name?)
 - script generation: Ingo_Script
 - script activation (e.g. upload to server): Ingo_Driver

But I think that makes perfect sense.

--
Brent J. Nordquist <bjn at horde.org> N0BJN
Other contact information: http://www.nordist.net/contact.html
* Fast pipe * Always on * Get out of the way - Tim Bray
http://tinyurl.com/7sti



----- End forwarded message -----


______________________________________________
Michael Slusarz [slusarz at bigworm.colorado.edu]
The University of Colorado at Boulder