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

Re: r20600 broke mailer.py's diff url promises

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: 2006-07-25 17:41:31 CEST

Blair Zajac wrote:
> C. Michael Pilato wrote:
>> mailer.py's sample configuration file -- and, until r20600, behavior --
>> gives the following expected behavior for the diff_*_url options:
>> # Diff URL construction. For the configured diff URL types, the diff
>> # section (which follows the message header) will include the URL
>> # relevant to the change type, even if actual diff generation for that
>> # change type is disabled (per the generate_diffs option).
>> #
>> # Available substitution variables are: path, base_path, rev, base_rev
>> But as of r20600, those substitution variables (path, base_path, rev,
>> base_rev) no longer work.
>> I haven't really jumped into a debugging session, but my guess is that
>> the
>> substitution is now happening earlier than it was before (in the calls to
>> cfg.get()), specifically, too early for the above variables to have been
>> defined.
>> I'm tracking this regression in issue #2587.
> So r20600 worked for me because I was using mappings in our production
> environment, which differed the parameter substitution till later, hence
> I didn't see the bug.
> The below patch, which I can apply when people are happy with it, defers
> the parameter retrieval.
> I tested this patch with a mailer.conf that uses mapping and one that
> does not for the diff_*_url values.

I feel good about your patch, because it is almost exactly like the one I
have in progress here. I made a minor misstep in mine, though, that yours
doesn't make. I'll apply and test your patch a little later today, after my
morning mail slog. Thanks, Blair.

C. Michael Pilato <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on Tue Jul 25 17:42:23 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.