Merge lp:~ubuntu-mate-dev/yelp/mate-compatibility into lp:~ubuntu-desktop/yelp/ubuntu

Proposed by Martin Wimpress 
Status: Merged
Merged at revision: 138
Proposed branch: lp:~ubuntu-mate-dev/yelp/mate-compatibility
Merge into: lp:~ubuntu-desktop/yelp/ubuntu
Diff against target: 25 lines (+2/-2)
2 files modified
debian/control (+1/-1)
debian/control.in (+1/-1)
To merge this branch: bzr merge lp:~ubuntu-mate-dev/yelp/mate-compatibility
Reviewer Review Type Date Requested Status
Ubuntu Sponsors Pending
Ubuntu Desktop Pending
Review via email: mp+297171@code.launchpad.net

Description of the change

mate-user-guide requires Yelp. Yelp recommends gnome-user-guide, so Ubuntu MATE ends up with gnome-user-guide being pointlessly installed.

Xubuntu seed Yelp and don't require gnome-user-guide nor mate-user-guide. This merge proposal make gnome-user-guide or mate-user-guide Suggests to prevent unnecessary installs of either user-guide.

To post a comment you must log in.
Revision history for this message
Jeremy Bícha (jbicha) wrote :

I just want to mention that the Help button in gnome-disk-utility points to the gnome-user-guide. If MATE provided documentation for that, then maybe it should conditionally point to MATE's documentation when run in MATE. Alternatively, you could ask gnome-disk-utility to ship its own documentation, at least for the benefit of other desktop environments.

The same problem happens if a MATE user installs Nautilus (although MATE uses caja by default as file browser).

$ apt show gnome-user-guide
Version: 3.20.2-1
Installed-Size: 47.3 MB
Download-Size: 2,361 kB

$ apt show ubuntu-docs
Version: 16.04.3
Installed-Size: 67.4 MB
Download-Size: 1,152 kB

Anyway, I'm not blocking this but just wanted you to be aware of this.

139. By Martin Wimpress 

Update debian/control.in accordingly.

Revision history for this message
Martin Pitt (pitti) wrote :

Martin, please include debian/changelog records in your merges. It's very inconvenient to have to add them, even more so as you don't have a published email address so it's not trivial to make up a real changelog record.

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 2016-05-17 01:50:47 +0000
3+++ debian/control 2016-06-23 08:24:00 +0000
4@@ -42,7 +42,7 @@
5 libyelp0 (= ${binary:Version}),
6 yelp-xsl (>= 3.12.0),
7 man-db (>= 2.5.1-1)
8-Recommends: gnome-user-guide
9+Suggests: gnome-user-guide | mate-user-guide
10 Description: Help browser for GNOME
11 Yelp is the help browser for the GNOME desktop. Yelp provides a simple
12 graphical interface for viewing DocBook, Mallard, HTML, man, and info
13
14=== modified file 'debian/control.in'
15--- debian/control.in 2016-05-17 01:50:47 +0000
16+++ debian/control.in 2016-06-23 08:24:00 +0000
17@@ -40,7 +40,7 @@
18 libyelp0 (= ${binary:Version}),
19 yelp-xsl (>= 3.12.0),
20 man-db (>= 2.5.1-1)
21-Recommends: gnome-user-guide
22+Suggests: gnome-user-guide | mate-user-guide
23 Description: Help browser for GNOME
24 Yelp is the help browser for the GNOME desktop. Yelp provides a simple
25 graphical interface for viewing DocBook, Mallard, HTML, man, and info

Subscribers

People subscribed via source and target branches