[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[RFR] templates://postgrey/{postgrey.templates}



Please find, for review, the debconf templates and packages descriptions for the postgrey source package.

This review will last from Friday, March 06, 2009 to Monday, March 16, 2009.

Please send reviews as unified diffs (diff -u) against the original
files. Comments about your proposed changes will be appreciated.

Your review should be sent as an answer to this mail.

When appropriate, I will send intermediate requests for review, with
"[RFRn]" (n>=2) as a subject tag.

When we will reach a consensus, I send a "Last Chance For
Comments" mail with "[LCFC]" as a subject tag.

Finally, the reviewed templates will be sent to the package maintainer
as a bug report, and a mail will be sent to this list with "[BTS]" as
a subject tag.

Rationale:
--- postgrey.old/debian/postgrey.templates	2009-03-02 09:00:55.683431768 +0100
+++ postgrey/debian/postgrey.templates	2009-03-06 07:42:07.166959821 +0100
@@ -1,6 +1,8 @@
 Template: postgrey/1.32-3_changeport
 Type: note
 _Description: Default TCP port change
- Postgrey is now listening to port 10023 and not 60000.  The latter was an
- unnecessary Debian specific modification.  You will need to adjust your
+ Postgrey is now listening to port 10023 and not 60000. That will
+ bring the software behavior closer to the default upstream settings.
+ .
+ You will need to adjust postgrey's
  configuration (usually /etc/postfix/main.cf) accordingly.

Avoid "branding" with Debian, which makes the package more friendly to
derivative distributions. That also avoids the apparent judgement on
pevious settings....:-)


--- postgrey.old/debian/control	2009-03-02 09:00:55.683431768 +0100
+++ postgrey/debian/control	2009-03-06 07:44:38.678960371 +0100
@@ -15,11 +15,12 @@
 Recommends: libdigest-sha1-perl, libnet-rblclient-perl, libparse-syslog-perl,
  postfix
 Description: greylisting implementation for Postfix
- A policy server for Postfix implementing greylisting.
+ This package provides a policy server for Postfix, that implements "greylisting".

Make a full sentence with the common wording "This package provides <foo>"

  .
- Greylisting means that you reject email from a server on the first try,
- using the fact that most spammers do not retry to send their email,
+ Greylisting is a spam filtering method that first rejects email coming
+ from external servers on the first try. It is based on the assumption that
+ most spammers do not retry to send their junk
  whereas almost all normal mail servers do.

"you" do not reject mails..:-)... If we're strict, the method doesn't
reject mails either, indeed....that's the job of the software..:-)

spammers do not send mails but junk (OK, *that* is non neutral....but
we all hate spammers, don't we?)

Could be "a method where mails are rejected on the first try"....

  .
  While Postgrey is designed for the use with Postfix, it can also be used
- with Exim.
+ with Exim or other mail transfer agents.

I'm not sure that my assumption is correct but isn't there a
possibility that postgrey also works with other MTA?

-- 


Template: postgrey/1.32-3_changeport
Type: note
_Description: Default TCP port change
 Postgrey is now listening to port 10023 and not 60000. That will
 bring the software behavior closer to the default upstream settings.
 .
 You will need to adjust postgrey's
 configuration (usually /etc/postfix/main.cf) accordingly.
--- postgrey.old/debian/postgrey.templates	2009-03-02 09:00:55.683431768 +0100
+++ postgrey/debian/postgrey.templates	2009-03-06 07:42:07.166959821 +0100
@@ -1,6 +1,8 @@
 Template: postgrey/1.32-3_changeport
 Type: note
 _Description: Default TCP port change
- Postgrey is now listening to port 10023 and not 60000.  The latter was an
- unnecessary Debian specific modification.  You will need to adjust your
+ Postgrey is now listening to port 10023 and not 60000. That will
+ bring the software behavior closer to the default upstream settings.
+ .
+ You will need to adjust postgrey's
  configuration (usually /etc/postfix/main.cf) accordingly.
--- postgrey.old/debian/control	2009-03-02 09:00:55.683431768 +0100
+++ postgrey/debian/control	2009-03-06 07:44:38.678960371 +0100
@@ -15,11 +15,12 @@
 Recommends: libdigest-sha1-perl, libnet-rblclient-perl, libparse-syslog-perl,
  postfix
 Description: greylisting implementation for Postfix
- A policy server for Postfix implementing greylisting.
+ This package provides a policy server for Postfix, that implements "greylisting".
  .
- Greylisting means that you reject email from a server on the first try,
- using the fact that most spammers do not retry to send their email,
+ Greylisting is a spam filtering method that first reject email coming
+ from external servers on the first try. It is based on the assumption that
+ most spammers do not retry to send their junk
  whereas almost all normal mail servers do.
  .
  While Postgrey is designed for the use with Postfix, it can also be used
- with Exim.
+ with Exim or other mail transfer agents..
Source: postgrey
Section: mail
Priority: optional
Maintainer: Adrian von Bidder <cmot@debian.org>
Build-Depends: debhelper (>= 4.1.46), po-debconf
Build-Depends-Indep: quilt
Standards-Version: 3.8.0
Homepage: http://postgrey.schweikert.ch/

Package: postgrey
Architecture: all
Depends: adduser, debconf, libberkeleydb-perl (>= 0.23-2), libnet-dns-perl,
 libnet-server-perl (>= 0.87), perl, ucf (>= 0.28)
Conflicts: libsys-syslog-perl (<< 0.21)
Recommends: libdigest-sha1-perl, libnet-rblclient-perl, libparse-syslog-perl,
 postfix
Description: greylisting implementation for Postfix
 This package provides a policy server for Postfix, that implements "greylisting".
 .
 Greylisting is a spam filtering method that first rejects email coming
 from external servers on the first try. It is based on the assumption that
 most spammers do not retry to send their junk
 whereas almost all normal mail servers do.
 .
 While Postgrey is designed for the use with Postfix, it can also be used
 with Exim or other mail transfer agents.

Attachment: signature.asc
Description: Digital signature


Reply to: