[exim-cvs] Minor doc nits re bug 1262.

Gitweb: http://git.exim.org/exim.git/commitdiff/8b26070532a925dd0a2c1c9d0deb0894a4af8eec

Commit:     8b26070532a925dd0a2c1c9d0deb0894a4af8eec
Parent:     ea7224901573939b2aa159db5517c4ab60e22539
Author:     Phil Pennock <pdp@???>
AuthorDate: Tue Sep 11 20:14:42 2012 -0400
Committer:  Phil Pennock <pdp@???>
CommitDate: Tue Sep 11 20:14:42 2012 -0400

    Minor doc nits re bug 1262.

    Update src comment to be clearer about why it's safe for "state of this transport" to affect other deliveries.
    Mention change in externally observable state in README.UPDATING.
    Reference bugzilla entry in ChangeLog.
    Update Paul's credit in ACKNOWLEDGMENTS.
---
 doc/doc-txt/ChangeLog     |    3 ++-
 src/ACKNOWLEDGMENTS       |    1 +
 src/README.UPDATING       |    5 +++++
 src/src/transports/smtp.c |    4 ++--
 4 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog
index f9eaaf7..a4c10d1 100644
--- a/doc/doc-txt/ChangeLog
+++ b/doc/doc-txt/ChangeLog
@@ -69,7 +69,8 @@ JH/05 Permit multiple router/transport headers_add/remove lines.
JH/06 Add dnsdb pseudo-lookup "a+" to do an "aaaa" + "a" combination.

 JH/07 Avoid using a waiting database for a single-message-only transport.
-      Performance patch from Paul Fisher.
+      Performance patch from Paul Fisher.  Bugzilla 1262.
+

 Exim version 4.80
 -----------------
diff --git a/src/ACKNOWLEDGMENTS b/src/ACKNOWLEDGMENTS
index a611a29..a66512f 100644
--- a/src/ACKNOWLEDGMENTS
+++ b/src/ACKNOWLEDGMENTS
@@ -382,6 +382,7 @@ Maxim Dounin              Patch portability of accept() len
 Frank Elsner              Fixed build reliability by exporting LC_ALL=C
 Paul Fisher               Diagnosed smtp_cmd_buffer_size affecting GSSAPI SASL
                             initial response, raised buffer size
+                          Patch adjusting connection_max_messages wait-DB usage
 Oliver Fleischmann        Patches fixing compilation in older compilers
 Julian Gilbey             Helped improve userforward local_part_suffix docs
 Richard Godbee            Patch fixing usage fprintf
diff --git a/src/README.UPDATING b/src/README.UPDATING
index b7406f4..5742a64 100644
--- a/src/README.UPDATING
+++ b/src/README.UPDATING
@@ -32,6 +32,11 @@ Exim version 4.81
  * New option gnutls_enable_pkcs11 defaults false; if you have GnuTLS 2.12.0
    or later and do want PKCS11 modules to be autoloaded, then set this option.

+ * A per-transport wait-<name> database is no longer updated if the transport
+ sets "connection_max_messages" to 1, as it can not be used and causes
+ unnecessary serialisation and load. External tools tracking the state of
+ Exim by the hints databases may need modification to take this into account.
+

Exim version 4.80
-----------------
diff --git a/src/src/transports/smtp.c b/src/src/transports/smtp.c
index 36a053f..dc24e69 100644
--- a/src/src/transports/smtp.c
+++ b/src/src/transports/smtp.c
@@ -3115,8 +3115,8 @@ for (addr = addrlist; addr != NULL; addr = addr->next)
for which hosts to become available. For some message-specific errors, the
update_waiting flag is turned off because we don't want follow-on deliveries in
those cases. If this transport instance is explicitly limited to one message
-per connection, follow-on deliveries are not possible, and there's no need
-to create/update a waiting database. */
+per connection then follow-on deliveries are not possible and there's no need
+to create/update the per-transport wait-<transport_name> database. */

if (update_waiting && tblock->connection_max_messages != 1)
transport_update_waiting(hostlist, tblock->name);