From: Phil Pennock Date: Wed, 12 Sep 2012 00:14:42 +0000 (-0400) Subject: Minor doc nits re bug 1262. X-Git-Tag: exim-4_81_RC1~3^2~71 X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/8b26070532a925dd0a2c1c9d0deb0894a4af8eec 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. --- diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index f9eaaf7a9..a4c10d13a 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 a611a291b..a66512f36 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 b7406f43c..5742a64b9 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- 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 36a053f2c..dc24e6938 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- database. */ if (update_waiting && tblock->connection_max_messages != 1) transport_update_waiting(hostlist, tblock->name);