[Mailman-developers] First patch -

Ken Manheimer klm@python.org
Tue, 14 Apr 1998 18:10:28 -0400 (EDT)


Here's the first mailman 1.0b2 patch.  For messages going to the
maillist, a blank line between the message headers and the body was
lacking, so if the first line of the body looked like a header (eg,
'http://www.python.org/'!), it would be included among the headers.

Here's the checkin message and the patch.  You need to apply the patch 
from the modules directory, if you use 'patch'.

Ken

.DeliverToList(): Was missing a newline between message headers and
body, so initial body lines that looked like header would be
incorporated in header.


*** mm_deliver.py.1.18	Tue Apr 14 18:01:34 1998
--- mm_deliver.py	Tue Apr 14 18:01:34 1998
***************
*** 122,138 ****
  	    msg.headers.append('To: %s\n' % self.GetListEmail())
   	else:
              tempfile.template = tmpfile_prefix + 'mailman.'
  	msg.headers.append('Errors-To: %s\n' % self.GetAdminEmail())
  
          tmp_file_name = tempfile.mktemp()
   	tmp_file = open(tmp_file_name, 'w+')
  
!  	tmp_file.write(string.join(msg.headers,''))
! 	# If replys don't go to the list, then they should go to the
! 	# real sender
! 	if self.reply_goes_to_list:
! 	    tmp_file.write('Reply-To: %s\n\n' % self.GetListEmail())
! 	if header:
  	    tmp_file.write(header + '\n')
  	tmp_file.write(self.QuotePeriods(msg.body))
  	if footer:
--- 122,136 ----
  	    msg.headers.append('To: %s\n' % self.GetListEmail())
   	else:
              tempfile.template = tmpfile_prefix + 'mailman.'
+ 	if self.reply_goes_to_list:
+             msg.headers.append('Reply-To: %s\n' % self.GetListEmail())
  	msg.headers.append('Errors-To: %s\n' % self.GetAdminEmail())
  
          tmp_file_name = tempfile.mktemp()
   	tmp_file = open(tmp_file_name, 'w+')
+  	tmp_file.write(string.join(msg.headers,'') + '\n')
  
! 	if header:                      # The *body* header:
  	    tmp_file.write(header + '\n')
  	tmp_file.write(self.QuotePeriods(msg.body))
  	if footer: