Merge lp://staging/~quickly-committers/quickly/702153 into lp://staging/quickly

Proposed by Michael Terry
Status: Needs review
Proposed branch: lp://staging/~quickly-committers/quickly/702153
Merge into: lp://staging/quickly
Diff against target: 89 lines (+48/-3)
3 files modified
quickly/builtincommands.py (+10/-1)
quickly/templatetools.py (+23/-2)
quickly/test/quickly.sh (+15/-0)
To merge this branch: bzr merge lp://staging/~quickly-committers/quickly/702153
Reviewer Review Type Date Requested Status
Tony Byrne Needs Fixing
Review via email: mp+67048@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Tony Byrne (tony-badwolf) wrote :

Hi Michael
 I looked at this quite a while ago and couldn't make up my mind if is was a bug or not. We will agree it is a bug because it generates a false error message.

 If the derived template was produced by an earlier version of quickly we would like to call upgrade on the project. Quickly marks *all* projects for upgrade that are created from derived templates, probably because it is difficult to discover which version of quickly produced the template.

templatetools contains the comment
# initialize with an empty project version to force first upgrade

Calling upgrade fails for templates produced by version 11.04 because it expects old code to be present in the project. I added a patch to https://bugs.launchpad.net/quickly/+bug/702153 which seems to directly address the problem.

I've marked this branch as needs fixing because I suspect that it will skip upgrade for old templates in ~/quickly-templates. I don't have a codebase to be sure.

review: Needs Fixing
Revision history for this message
Michael Terry (mterry) wrote :

OK, I applied your patch from the bug. I guess that means this branch is no longer quite needed? I can close it out if so.

Unmerged revisions

622. By Michael Terry

when creating a project, record version of parent templates

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches