On Tue, 22 Apr 2008 12:17:24 -0400 The Anarcat <[EMAIL PROTECTED]> wrote:
> On Tue, Apr 22, 2008 at 02:22:58PM +0300, Ion-Mihai Tetcu wrote: > > On Mon, 21 Apr 2008 13:30:03 -0400 > > The Anarcat <[EMAIL PROTECTED]> wrote: > > > > > On Mon, Apr 21, 2008 at 09:55:37AM +0300, Ion-Mihai Tetcu wrote: [ .. ] > > > > - it brings in unneeded overhead (dir/file reading) > > > > > > overhead?? how much overhead is it, actually, to read one directory and > > > its included files? especially since it's just done once, upon startup, > > > i think it's negligible overhead. > > > > If using --demon; else it's done each time. (I agree is negligible > > compared to the rest of things dspam is doing). > > Indeed, I didn't think of that use case. Then again, I can't think of > any reasonably sized deployment that wouldn't run dspam as --daemon... [ .. ] And why should dspam require "reasonably sized deployment" to be used? ;-) There are enough people out there using them from maildrop or equivalent for their personal mail for example, without bothering to use it as deamon. As a side note, it seems that more and more we, IT professionals, forget that 0.000something > 0 and that in enough places neither hardware or bandwidth is cheap. And when you add those 0.00something it all ends up to 'why optimize the code, let the user buy bigger better hardware' which ..... For example I have a user aboard on a boat; waterproof, fireproof, low heat hardware ain't cheap; bandwidth via satellite ain't cheap either. As a second example I ran dspam on a k6-II with 128MB RAM for about two years (mysql was on an other machine). > > > As for "working for the rest of us", I don't see how the patch breaks > > > anything for "the rest of us"... > > > > :-) > > > > Let's hide it behind a configure switch or OS check then; at very least > > it needs to respect PREFIX (sysconfdir) - which the current patch > > doesn't since it has that /etc hard-coded in it; having I'd really like to > > avoid having any of us maintain local patches to make it work right on > > our particular OS. > > Have it > > + Include @sysconfdir@/dspam/dspam.d/ > > _if_ that dir exists should be enough. > > > > Does this sound OK to you? > > That would be excellent. Besides, I was mostly jumping the gun and don't > have anything constructive to bring in besides trying to defend the > debian patchset.. ;) ;-) I'm glad you were defending it because it was needed and not because it was debian :) -- IOnut - Un^d^dregistered ;) FreeBSD "user" "Intellectual Property" is nowhere near as valuable as "Intellect" FreeBSD committer -> [EMAIL PROTECTED], PGP Key ID 057E9F8B493A297B
signature.asc
Description: PGP signature
