Merge lp:~dktrkranz/synaptic/manual into lp:synaptic

Proposed by Luca Falavigna
Status: Merged
Merged at revision: 1798
Proposed branch: lp:~dktrkranz/synaptic/manual
Merge into: lp:synaptic
Diff against target: 35 lines (+7/-1)
2 files modified
common/rpackageview.cc (+2/-1)
debian/changelog (+5/-0)
To merge this branch: bzr merge lp:~dktrkranz/synaptic/manual
Reviewer Review Type Date Requested Status
Michael Vogt Pending
Review via email: mp+42026@code.launchpad.net

Description of the change

* common/rpackageview.cc:
  - Do not display base packages in manual view.

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

Thanks for your branch.

I have to say that I'm a bit in split minds about it. Technically "Install (manual)" is no longer quite correct with it. But I see the value in protecting users from removing important stuff and apt will also not garbage collect stuff in manual mode.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'common/rpackageview.cc'
2--- common/rpackageview.cc 2010-05-04 13:53:00 +0000
3+++ common/rpackageview.cc 2010-11-28 00:25:41 +0000
4@@ -151,7 +151,8 @@
5 }
6
7 if ((flags & RPackage::FInstalled) &&
8- !(flags & RPackage::FIsAuto))
9+ !(flags & RPackage::FIsAuto) &&
10+ !(flags & RPackage::FImportant))
11 {
12 str = _("Installed (manual)");
13 _view[str].push_back(pkg);
14
15=== modified file 'debian/changelog'
16--- debian/changelog 2010-11-23 16:43:38 +0000
17+++ debian/changelog 2010-11-28 00:25:41 +0000
18@@ -1,5 +1,6 @@
19 synaptic (0.70~pre3) UNRELEASED; urgency=low
20
21+ [ Michael Vogt ]
22 * gtk/rgmainwindow.cc:
23 - show message in quick search label if no apt-xapian-index can
24 be found
25@@ -7,6 +8,10 @@
26 * po/hu.po:
27 - updated, thanks to Gabor Kum
28
29+ [ Luca Falavigna ]
30+ * common/rpackageview.cc:
31+ - Do not display base packages in manual view.
32+
33 -- Michael Vogt <michael.vogt@ubuntu.com> Wed, 10 Nov 2010 19:44:31 +0100
34
35 synaptic (0.70~pre2) unstable; urgency=low

Subscribers

People subscribed via source and target branches

to status/vote changes: