charm push does not accept merge revsions

Bug #1655051 reported by Aaron Bentley
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
charm (Ubuntu)
Confirmed
Undecided
Unassigned

Bug Description

charm push does not accept revisions produced by bzr merge. It complains:
ERROR cannot parse bzr log: cannot parse bzr log entry: invalid revision number "80 [merge]"

This persists even if a second, non merge commit is made. I.e. it cannot be recovered without changing commit history.

This is a regression in the sense that the old publication method (just pushing to Launchpad) had no problem with merge commits.

$ charm push . ~juju-qa/trusty/juju-reports
url: cs:~juju-qa/trusty/juju-reports-31
channel: unpublished
ERROR cannot parse bzr log: cannot parse bzr log entry: invalid revision number "80 [merge]"
------------------------------------------------------------
revno: 80 [merge]
revision-id: <email address hidden>
parent: <email address hidden>
parent: <email address hidden>
committer: Aaron Bentley <email address hidden>
branch nick: trunk
timestamp: Mon 2017-01-09 09:29:41 -0500
message:
  Add test-failure-email support to charm.
<snip>
$ bzr commit -m "No-change commit to give charm push a non-merge revno." --unchanged
Committing to: /home/abentley/canonical/repo/trusty/juju-reports/.bzr/branches/trunk/

You need a passphrase to unlock the secret key for
user: "Aaron Bentley <email address hidden>"
2048-bit RSA key, ID 9C7FEF61, created 2009-05-14

Committed revision 81.
$ bzr push
Using saved push location: bzr+ssh://bazaar.launchpad.net/~juju-qa/charms/trusty/juju-reports/trunk/
Pushed up to revision 81.
$ charm push . ~juju-qa/trusty/juju-reports
url: cs:~juju-qa/trusty/juju-reports-31
channel: unpublished
ERROR cannot parse bzr log: cannot parse bzr log entry: invalid revision number "80 [merge]"
------------------------------------------------------------
revno: 81
revision-id: <email address hidden>
parent: <email address hidden>
committer: Aaron Bentley <email address hidden>
branch nick: trunk
timestamp: Mon 2017-01-09 09:41:03 -0500
message:
  No-change commit to give charm push a non-merge revno.
------------------------------------------------------------
revno: 80 [merge]
revision-id: <email address hidden>
parent: <email address hidden>
parent: <email address hidden>
committer: Aaron Bentley <email address hidden>
branch nick: trunk
timestamp: Mon 2017-01-09 09:29:41 -0500
message:
  Add test-failure-email support to charm.
<snip>

Tags: regression
Revision history for this message
Aaron Bentley (abentley) wrote :

This is actually a real regression. 2.2.0-0ubuntu1~ubuntu16.04.1~ppa2 from the juju/stable PPA exhibits the bug, but charm 2.1.1-0ubuntu1 does not.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in charm (Ubuntu):
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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