Unity Asset Pool

Merge lp://qastaging/~ken-vandine/unity-asset-pool/0.8.24daily13.03.20.1-0ubuntu2 into lp://qastaging/unity-asset-pool

  1. 0.8.24daily13.03.20.1-0ubuntu2
  2. Merge into trunk
Proposed by Ken VanDine
Status: Rejected
Rejected by: Sebastien Bacher
Proposed branch: lp://qastaging/~ken-vandine/unity-asset-pool/0.8.24daily13.03.20.1-0ubuntu2
Merge into: lp://qastaging/unity-asset-pool
Diff against target: 35 lines (+8/-9)
2 files modified
debian/changelog (+8/-0)
debian/control (+0/-9)
To merge this branch: bzr merge lp://qastaging/~ken-vandine/unity-asset-pool/0.8.24daily13.03.20.1-0ubuntu2
Reviewer Review Type Date Requested Status
Sebastien Bacher Disapprove
Review via email: mp+154495@code.qastaging.launchpad.net

Commit message

* debian/control
    - Don't use a replaces, only breaks for the account-plugin-*
      packages (LP: #1157747)

Description of the change

- Don't use a replaces, only breaks for the account-plugin-* packages (LP: #1157747)

To post a comment you must log in.
lp://qastaging/~ken-vandine/unity-asset-pool/0.8.24daily13.03.20.1-0ubuntu2 updated
74. By Ken VanDine

fixed changelog

Revision history for this message
Sebastien Bacher (seb128) wrote :

thanks Ken but that's wrong, "Replaces:" is used for "can overwrite packages that were shipped in" and that's the right thing to do here

review: Disapprove

Unmerged revisions

74. By Ken VanDine

fixed changelog

73. By Ken VanDine

* debian/control
  - Don't use a breaks, only replaces for the account-plugin-*
    packages (LP: #1157747)

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

to all changes: