[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Serious terminology confusion in r20600

From: Max Bowsher <maxb1_at_ukf.net>
Date: 2006-07-13 17:03:20 CEST

Max Bowsher wrote:
> I just went to review r20600 for backport, and found serious terminology
> confusion - namely, the change doesn't actually have anything to do with
> what the log message describes.
>
> mailer.py has two separate concepts by which option values are processed:
>
> * 'maps': Defined in the [maps] section, and operate as lookup tables.
> * 'parameters': Defined by predefinition and regexp matches, and
> operate as string substitutions.
>
> The code change in r20600 actually enables parameter substitution for
> several additional parameters - which is a good change.
>
> However, the log message repeatedly refers to 'maps' and 'mappings',
> even quoting a comment from mailer.conf.example referring to mappings as
> justification for the change.
>
>
> I like the change, and am +1 on backporting the code to 1.4.x, but the
> log message and STATUS metadata are highly misleading, and should be
> fixed first.

On the other hand, with a RC about to happen, I'll do STATUS and the
merge, but the log message still needs fixing.

Max.

Received on Thu Jul 13 17:04:01 2006

This is an archived mail posted to the Subversion Dev mailing list.