Merge lp:~codygarver/pantheon-photos/conflicts-replace-shotwell into lp:~elementary-os/pantheon-photos/deb-packaging

Proposed by Cody Garver
Status: Merged
Approved by: Danielle Foré
Approved revision: 5
Merged at revision: 4
Proposed branch: lp:~codygarver/pantheon-photos/conflicts-replace-shotwell
Merge into: lp:~elementary-os/pantheon-photos/deb-packaging
Diff against target: 23 lines (+4/-2)
1 file modified
debian/control (+4/-2)
To merge this branch: bzr merge lp:~codygarver/pantheon-photos/conflicts-replace-shotwell
Reviewer Review Type Date Requested Status
Sergey "Shnatsel" Davidoff (community) Approve
Review via email: mp+220549@code.launchpad.net

Commit message

Conflicts and replace shotwell

Description of the change

The rebrand has landed and it was as nonintrusive as possible. Because of its nonintrusiveness, we must conflict with regular shotwell.

To post a comment you must log in.
4. By Cody Garver

Conflict with shotwell

Revision history for this message
Sergey "Shnatsel" Davidoff <unprivileged account> (shnatsel-unprivileged) wrote :

I'm pretty sure this requires "breaks" not "conflicts" as per Debian policy (or at least that's what lintian yells when I switch it to pedantic mode), but that might not play well with dpkg, e.g. it might not suggest removal of shotwell package. That needs testing though.

Revision history for this message
Cody Garver (codygarver) wrote :

Changing Conflicts to Breaks results in

codygarver@asus:/tmp/tmp.TgkWw4fMIY/p$ sudo dpkg -i ../pantheon-photos*.deb
[sudo] password for codygarver:
Selecting previously unselected package pantheon-photos.
dpkg: regarding .../pantheon-photos_0.17_amd64.deb containing pantheon-photos:
 pantheon-photos breaks shotwell
  shotwell (version 0.18.0-0ubuntu4) is present and installed.

dpkg: error processing archive ../pantheon-photos_0.17_amd64.deb (--install):
 installing pantheon-photos would break shotwell, and
 deconfiguration is not permitted (--auto-deconfigure might help)
Selecting previously unselected package pantheon-photos-common.
dpkg: regarding .../pantheon-photos-common_0.17_all.deb containing pantheon-photos-common:
 pantheon-photos-common breaks shotwell-common
  shotwell-common (version 0.18.0-0ubuntu4) is present and installed.

dpkg: error processing archive ../pantheon-photos-common_0.17_all.deb (--install):
 installing pantheon-photos-common would break shotwell-common, and
 deconfiguration is not permitted (--auto-deconfigure might help)
Errors were encountered while processing:
 ../pantheon-photos_0.17_amd64.deb
 ../pantheon-photos-common_0.17_all.deb

5. By Cody Garver

Cleanup

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Yeah, that's what I expected. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2014-02-24 07:11:46 +0000
3+++ debian/control 2014-05-23 07:43:05 +0000
4@@ -35,6 +35,8 @@
5
6 Package: pantheon-photos
7 Architecture: linux-any
8+Conflicts: shotwell
9+Replaces: shotwell
10 Depends: ${shlibs:Depends},
11 ${misc:Depends},
12 pantheon-photos-common (= ${source:Version}),
13@@ -52,8 +54,8 @@
14 Architecture: all
15 Depends: ${misc:Depends}
16 Recommends: pantheon-photos
17-Breaks: shotwell (<< 0.14.1-1)
18-Replaces: shotwell (<< 0.14.1-1)
19+Conflicts: shotwell-common
20+Replaces: shotwell-common
21 Description: digital photo organizer - common files
22 Photos is a digital photo organizer designed for elementary's Pantheon desktop
23 environment. It allows you to import photos from disk or camera, organize

Subscribers

People subscribed via source and target branches

to all changes: