Ubuntu Software Center

Merge lp:~ilidrissi.amine/software-center/addons-no-recommends into lp:software-center

  1. addons-no-recommends
  2. Merge into trunk
Proposed by Mohamed Amine Ilidrissi
Status: Rejected
Rejected by: Michael Vogt
Proposed branch: lp:~ilidrissi.amine/software-center/addons-no-recommends
Merge into: lp:software-center
Diff against target: 59 lines (+18/-6) (has conflicts)
2 files modified
debian/changelog (+9/-0)
softwarecenter/apt/aptcache.py (+9/-6)
Text conflict in debian/changelog
To merge this branch: bzr merge lp:~ilidrissi.amine/software-center/addons-no-recommends
Reviewer Review Type Date Requested Status
Matthew Paul Thomas Disapprove
Review via email: mp+34295@code.launchpad.net

Description of the change

Recommended add-ons will not be shown when the app is not installed, since they are automatically installed.

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

A good example to test this is gnumeric

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

To me, Gnumeric looks like an example of why this would be undesirable. gnumeric Recommends: evince | evince-gtk, lp-solve, and gnumeric-doc. And you might reasonably want to install Gnumeric without any of those three. (It seems lp-solve currently isn't displayed only because openoffice.org-calc Depends: on it.)

Either Recommends: is just a synonym for Depends:, or it isn't. If it is, then it will seem redundant to new developers, making packaging unduly complicated. If it isn't, then we can report and fix inappropriate Recommends: values in existing packages, in the same way that we've been fixing bad package titles and descriptions.

review: Disapprove
Revision history for this message
Mohamed Amine Ilidrissi (ilidrissi.amine) wrote :

mpt, recommended packages are not shown already, I just thought they were shown, my bad :(

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

Recommended packages are shown for me in 2.1.22.3.

Revision history for this message
Michael Vogt (mvo) wrote :

Thanks for your work on this!

I still mark this as rejected for now as mpt disagrees with the design. Sorry for that and looking forward for more branches from you :)

Happy new year,
 Michael

Unmerged revisions

1094. By Mohamed Amine Ilidrissi

softwarecenter/apt/aptcache.py: Recommended add-ons will not be
shown when the app is not installed, since they are automatically
installed

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2010-09-01 14:47:12 +0000
+++ debian/changelog 2010-09-01 14:50:32 +0000
@@ -32,6 +32,7 @@
32 pathbar no longer shrinking to available width (LP: #625210),32 pathbar no longer shrinking to available width (LP: #625210),
33 many thanks Matthew McGowan!33 many thanks Matthew McGowan!
3434
35<<<<<<< TREE
35 [ Kiwinote ]36 [ Kiwinote ]
36 * data/dialogs.ui:37 * data/dialogs.ui:
37 - only display vscrollbar if needed for dependency removal alert38 - only display vscrollbar if needed for dependency removal alert
@@ -67,6 +68,14 @@
67 - don't show as 'untitled window' in taskbar68 - don't show as 'untitled window' in taskbar
6869
69 -- Michael Vogt <michael.vogt@ubuntu.com> Wed, 01 Sep 2010 09:50:07 +020070 -- Michael Vogt <michael.vogt@ubuntu.com> Wed, 01 Sep 2010 09:50:07 +0200
71=======
72 [ Mohamed Amine IL Idrissi ]
73 * softwarecenter/apt/aptcache.py: Recommended add-ons will not be
74 shown when the app is not installed, since they are automatically
75 installed
76
77 -- Mohamed Amine IL Idrissi <ilidrissiamine@gmail.com> Wed, 01 Sep 2010 13:42:51 +0000
78>>>>>>> MERGE-SOURCE
7079
71software-center (2.1.14.2) maverick; urgency=low80software-center (2.1.14.2) maverick; urgency=low
7281
7382
=== modified file 'softwarecenter/apt/aptcache.py'
--- softwarecenter/apt/aptcache.py 2010-08-31 15:47:29 +0000
+++ softwarecenter/apt/aptcache.py 2010-09-01 14:50:32 +0000
@@ -385,8 +385,10 @@
385 pkg = self._cache[pkgname]385 pkg = self._cache[pkgname]
386386
387 # recommended addons387 # recommended addons
388 addons_rec = self.get_recommends(pkg)388 addons_rec = []
389 LOG.debug("recommends: %s" % addons_rec)389 if pkg.installed:
390 addons_rec = self.get_recommends(pkg)
391 LOG.debug("recommends: %s" % addons_rec)
390 # suggested addons and renhances392 # suggested addons and renhances
391 addons_sug = self.get_suggests(pkg)393 addons_sug = self.get_suggests(pkg)
392 LOG.debug("suggests: %s" % addons_sug)394 LOG.debug("suggests: %s" % addons_sug)
@@ -416,10 +418,11 @@
416 pkgdep = self._cache[dep]418 pkgdep = self._cache[dep]
417 if len(self.get_rdepends(pkgdep)) == 1:419 if len(self.get_rdepends(pkgdep)) == 1:
418 # pkg is the only known package that depends on pkgdep420 # pkg is the only known package that depends on pkgdep
419 pkgdep_rec = self.get_recommends(pkgdep)421 if pkg.installed:
420 LOG.debug("recommends from lonley dependency %s: %s" % (422 pkgdep_rec = self.get_recommends(pkgdep)
421 pkgdep, pkgdep_rec))423 LOG.debug("recommends from lonley dependency %s: %s" % (
422 addons_rec += pkgdep_rec424 pkgdep, pkgdep_rec))
425 addons_rec += pkgdep_rec
423 pkgdep_sug = self.get_suggests(pkgdep)426 pkgdep_sug = self.get_suggests(pkgdep)
424 LOG.debug("suggests from lonley dependency %s: %s" % (427 LOG.debug("suggests from lonley dependency %s: %s" % (
425 pkgdep, pkgdep_sug))428 pkgdep, pkgdep_sug))

Subscribers

People subscribed via source and target branches