Merge lp://qastaging/~zyga/checkbox/checkbox-packaging into lp://qastaging/~checkbox-dev/checkbox/checkbox-packaging

Proposed by Zygmunt Krynicki
Status: Merged
Approved by: Daniel Manrique
Approved revision: no longer in the source branch.
Merged at revision: 35
Proposed branch: lp://qastaging/~zyga/checkbox/checkbox-packaging
Merge into: lp://qastaging/~checkbox-dev/checkbox/checkbox-packaging
Diff against target: 94 lines (+38/-11)
4 files modified
debian/checkbox-autostart-desktop.install (+1/-1)
debian/checkbox-autostart-server.install (+1/-1)
debian/checkbox-ci-mailer.install (+1/-1)
debian/control (+35/-8)
To merge this branch: bzr merge lp://qastaging/~zyga/checkbox/checkbox-packaging
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+194213@code.qastaging.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

This looks OK, just some renaming and moving files and a few transitional packages.

Do you want to remove the plainbox-autostart-* transitional packages now? I think they're only used in checkbox-
satellite deployments so as long as we push the corresponding changes today and alert everybody to upgrade, we should be OK. Or we can do it in a more staggered fashion.

+1 but I won't pull the approval trigger until I hear about this.

review: Approve
Revision history for this message
Daniel Manrique (roadmr) wrote :

ok, I'll approve this now, we can remove those packages later it's no rush :)

35. By Zygmunt Krynicki

"automatic merge by tarmac [r=roadmr][bug=][author=zkrynicki]"

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

On śro, lis 6, 2013 o 9:29 , Daniel Manrique
<email address hidden> wrote:
> Review: Approve
>
> This looks OK, just some renaming and moving files and a few
> transitional packages.
>
> Do you want to remove the plainbox-autostart-* transitional packages
> now? I think they're only used in checkbox-
> satellite deployments so as long as we push the corresponding changes
> today and alert everybody to upgrade, we should be OK. Or we can do
> it in a more staggered fashion.
>

Not yet. We can look at removing them afterr a few weeks. I wonder what
the policy says on transitional packages like that.

Thanks
ZK
>
>

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