Launchpad itself

deleting revno from main branch does not work as expected

Bug #929938 reported by David Britton on 2012-02-10

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Launchpad itself	Fix Released	Low	William Grant

Sample Recipe:

# bzr-builder format 0.3 deb-version {debupstream}-0~bzr{revno}~landscape0~bzr{revno:packaging}
lp:commandant revno:HEAD
nest packaging lp:~landscape/commandant/packaging debian

I try taking the revno:HEAD out, and it fails, I can change it to another number, but I want it gone.

Tags:

Merged into lp:launchpad at revision 14788

Jelmer Vernooij (community): Approve on 2012-02-11

Revision history for this message

Aaron Bentley (abentley) wrote on 2012-02-10:

How does it fail? This recipe works in local testing for me:

# bzr-builder format 0.3 deb-version {debupstream}-0~bzr{revno}~landscape0~bzr{revno:packaging}
lp:commandant
nest packaging lp:~landscape/commandant/packaging debian

Changed in launchpad:
status:	New → Incomplete

Revision history for this message

William Grant (wgrant) wrote on 2012-02-11:

The issue is that it's impossible to unset the revspec of a SourcePackageRecipeData. setRecipe does this:

if builder_recipe.revspec is not None:
self.revspec = unicode(builder_recipe.revspec)

... which doesn't help when the revspec is desired to be None.

I believe that setting it to '-1' should be a reasonable workaround for now.

William Grant (wgrant) on 2012-02-11

Revision history for this message

Launchpad QA Bot (lpqabot) wrote on 2012-02-14:

William Grant (wgrant) on 2012-02-14

tags:

added: qa-ok
removed: qa-needstesting

William Grant (wgrant) on 2012-02-16

Changed in launchpad:
status:	Fix Committed → Fix Released

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Bug watches keep track of this bug in other bug trackers.