Barry A. Warsaw wrote:
"CT" == Christian Tismer <tismer@appliedbiometrics.com> writes:
CT> I will also propose to change Mailman to handle this CT> in a better way. My own patched version on Starship CT> never prepends the list name if it can be matched in the CT> "re" already.I thought Mailman already does this too. I'll double check. Chris, you might want to send your patches to mailman-developers@python.org
Well, I had a look: New Mailman does this:
prefix = self.subject_prefix
if not subj:
msg.SetHeader('Subject', '%s(no subject)' % prefix)
elif not re.match("(re:? *)?" + re.escape(self.subject_prefix),
subj, re.I):
msg.SetHeader('Subject', '%s%s' % (prefix, subj))
if self.anonymous_list:This is insufficient for cross-posts which are replied to from a different list. I think my string.find approach is crude, simple, but exactly the right thing. Forget about seldom possible missing prefixes...
ciao - chris
-- Christian Tismer :^) <mailto:tismer@appliedbiometrics.com> Applied Biometrics GmbH : Have a break! Take a ride on Python's Kaiserin-Augusta-Allee 101 : *Starship* http://starship.skyport.net 10553 Berlin : PGP key -> http://pgp.ai.mit.edu/ we're tired of banana software - shipped green, ripens at home