launchpad-reviewers team mailing list archive

[Gary Poster <gary.poster@xxxxxxxxxxxxx>]

The proposal to merge lp:~gary/launchpad/bug632218 into lp:launchpad has been updated.

Description changed to:

This is a release-critical branch to fix a critical bug.

The symptoms are described in the bug report.

LOSAs are hopefully going to verify the fix in production in a few hours.  Meanwhile, I plan to go ahead and try to get this merged, because I believe it will work.

To fix this problem, I took the approach of adding an explicit step for compiling the templates within make compile.  The cheapest yet most comprehensive way I could think of to do this was to process Launchpad's zcml.  This should load all of our normal templates in the future.  For now, it loads the Python file that registers the ++profile++ adapter and thus processes the template.  You could argue that this is a side effect, and it is, but a defensible one: if zcml registers a view, the view should be instantiated with its template in order to be registered.  If we ever want to allow for lazy template generation for some reason, we can deal with that then, and I don't see a big advantage for us anyway.

This also makes "make clean" clean up .pt.py files generated by chameleon.
-- 
https://code.launchpad.net/~gary/launchpad/bug632218/+merge/34785
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~gary/launchpad/bug632218 into lp:launchpad.