Merge lp:~evfool/software-properties/lp1060543 into lp:software-properties

Proposed by Robert Roth on 2012-11-21
Status: Merged
Merged at revision: 823
Proposed branch: lp:~evfool/software-properties/lp1060543
Merge into: lp:software-properties
Diff against target: 27 lines (+4/-3)
2 files modified
data/gtkbuilder/main.ui (+1/-1)
data/software-properties-gtk.desktop.in (+3/-2)
To merge this branch: bzr merge lp:~evfool/software-properties/lp1060543
Reviewer Review Type Date Requested Status
Didier Roche 2012-11-21 Approve on 2012-11-23
Review via email: mp+135513@code.launchpad.net

Description of the Change

Added some keywords to the software sources desktop file for easier discovery of the driver management functionality (search for Driver/Drivers shows software sources after adding Drivers as a keyword).

To post a comment you must log in.
Robert Roth (evfool) wrote :

I have updated the merge proposal with the following changes based on the feedback from the bug:
* copied the jockey desktop file with its icons from jockey trunk (it gets installed with software-properties-gtk)
* renamed it to additional-drivers.desktop
* removed the control center panel category
* updated the translation domain

Didier Roche (didrocks) wrote :

Hey, it seems that mpt has another opinion on how to fix it rather than adding another desktop file to it, see https://lists.ubuntu.com/archives/ubuntu-desktop/2012-November/004121.html (and the full discussion which is worth reading). Can you please rather implements that?

review: Needs Fixing
Robert Roth (evfool) wrote :

Could you please be a bit more explicit on what do you mean by "implement that"?
mpt's suggestions is about
* renaming Software sources to Software & updates (Easy enough)
* finding it by searching for "drivers" = adding a keyword to the desktop file (done)
* and the wiki spec details a control center panel (embedded), which as far as I know is not possible with the current python-based software sources, it would require porting the whole thing to C, and I can't do that, if that's the requirement here, I'll pass on to someone else.

Didier Roche (didrocks) wrote :

@Robert: I think mpt just wants the two bullet points to be addressed rathen than adding another desktop file, which means:
- renaming software sources
- adding the drivers keyword.

(and so, not adding a jockey desktop file).
Did I misunderstand him?

Robert Roth (evfool) wrote :

@didrocks: No, that's what he basically asks in his email, but he is also pointing to the spec, which describes a control center panel. Anyway, for now I have reverted the changes related to the new desktop file, and updated the Software Sources title to match the spec.

Didier Roche (didrocks) wrote :

Excellent! Pushing to trunk :)

Thanks a lot!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/gtkbuilder/main.ui'
2--- data/gtkbuilder/main.ui 2012-11-09 17:44:47 +0000
3+++ data/gtkbuilder/main.ui 2012-11-23 12:22:22 +0000
4@@ -93,7 +93,7 @@
5 <object class="GtkWindow" id="window_main">
6 <property name="can_focus">False</property>
7 <property name="border_width">6</property>
8- <property name="title" translatable="yes">Software Sources</property>
9+ <property name="title" translatable="yes">Software &amp; Updates</property>
10 <property name="window_position">center</property>
11 <signal name="delete-event" handler="on_delete_event" swapped="no"/>
12 <child>
13
14=== modified file 'data/software-properties-gtk.desktop.in'
15--- data/software-properties-gtk.desktop.in 2012-11-08 03:25:41 +0000
16+++ data/software-properties-gtk.desktop.in 2012-11-23 12:22:22 +0000
17@@ -1,7 +1,8 @@
18 [Desktop Entry]
19-_Name=Software Sources
20-_GenericName=Software Sources
21+_Name=Software & Updates
22+_GenericName=Software & Updates
23 _Comment=Configure the sources for installable software and updates
24+_Keywords=Drivers;Repositories;Repository;PPA;
25 Exec=software-properties-gtk
26 Icon=software-properties
27 Terminal=false

Subscribers

People subscribed via source and target branches

to status/vote changes: