[Bug 189188] Review Request: sqlgrey - postfix grey-listing policy service

bugzilla at redhat.com bugzilla at redhat.com
Mon Sep 18 21:15:46 UTC 2006


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

Summary: Review Request: sqlgrey - postfix grey-listing policy service


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





------- Additional Comments From steve at silug.org  2006-09-18 17:15 EST -------
(In reply to comment #8)
> 1. Might include the gpl COPYING file?
> 
> 2. Might include the Changelog and CONTRIB as %doc files?

I'll fix that in -3.

> 3. rpmlint says:
> 
> W: sqlgrey conffile-without-noreplace-flag /etc/sqlgrey/README
> 
> Does this need to be in /etc/sqlgrey? Or can it be in doc?

Here are the contents of that file:

  README file for sqlgrey updated content
  
  Don't touch these files, they are automatically updated when
  you run update_sqlgrey_config:
  - clients_fqdn_whitelist: don't greylist these DNS names [*]
  - clients_ip_whitelist  : don't greylist these IP addresses [*]
  - dyn_fqdn.regexp       : used by new 'smart' algorithm [x]
  - smtp_server.regexp    : used by new 'smart' algortihm [x]
  
  [*]: in use starting with 1.4.0
  [x]: in use since 1.5.1, regexps looking for known fqdns patterns

So I think it seems appropriate to leave it in that directory.

> E: sqlgrey non-standard-uid /var/lib/sqlgrey sqlgrey
> E: sqlgrey non-standard-gid /var/lib/sqlgrey sqlgrey
> E: sqlgrey non-standard-dir-perm /var/lib/sqlgrey 0750
> 
> All those can be ignored I think.

Agreed.

> 4. Have you been able to determine the issue from comment #6 about it not 
> working? 

Not yet, but until today I haven't taken a moment to look at it.

Once I'm done with my last couple of pre-FC6 package upgrades, I should have
some time for this again.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list