[Bug 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=537587





--- Comment #35 from Nathanael Noblet <nathanael at gnat.ca>  2009-12-03 15:33:19 EDT ---
I think I caught everything you pointed out except with the following:

1) I didn't do much about file/dir permissions yet but will. I have questions
I'll post in a separate comment.

2) I didn't rename the libdspam to dspam-libs because you can install the
library independent of dspam, it can be used in other projects/programs. If
someone were to write a plugin for evolution or thunderbird. the dspam package
is only necessary for an MTA, whereas the library can be used in all sorts of
places.

3) The libdspam-X are required by the library for storage of all the token data
it produces. So I'm not sure what to call them other than libdspam-X, I could
rename them to dspam-plugin-X or dspam-storage-X but that implies you need the
dspam package which you don't. So if you could advise what to do here.

%changelog
* Sun Nov 29 2009 Nathanael Noblet <nathanael at gnat.ca> - 3.9.0-0.10.BETA4
- Change dspam init to not be enabled by default
- Add EVR dependancy from all sub packages
- Removed pkgconfig requirement
- package -n dspam-web renamed to package web
- Updated perl requires even though dspam-web is not being built currently
- Added post/preun requirements to chkconfig and initscripts
- Updated defattr lines
- use _initrddir instead of _sysconfdir/rc.d/init.d (RHEL compat)
- updated source file permissions
- get libdspam to own the sql-scripts directory

http://www.gnat.ca/dspam.spec
http://www.gnat.ca/dspam-3.9.0-0.10.BETA4.fc12.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.