Code review comment for lp://staging/~gary/launchpad/bug559128

Revision history for this message
Gary Poster (gary) wrote :

On Apr 21, 2010, at 4:44 PM, Francis J. Lacoste wrote:

> Review: Approve
> Looks good. I suppose we will QA this by removing the work-around from edge deployment config and seeing if the edge update breaks?

Yeah. I also QA'd it by doing the following:

- (get branch and dependencies)
- make bin/buildout && bin/buildout
- cd ..
- mv (branch) (new location)
- cd (new location)
- make build
- make run

That worked. That's the closest thing I can figure on doing locally.

Gary

« Back to merge proposal