Merge lp:~hmollercl/software-properties/software-properties into lp:software-properties

Proposed by Hans P. Möller
Status: Merged
Merged at revision: 1078
Proposed branch: lp:~hmollercl/software-properties/software-properties
Merge into: lp:software-properties
Diff against target: 321 lines (+62/-36) (has conflicts)
7 files modified
data/designer/main.ui (+2/-2)
data/software-properties-drivers-qt.desktop.in (+13/-0)
data/software-properties-qt.desktop.in (+2/-2)
debian/changelog (+14/-0)
debian/control (+1/-1)
softwareproperties/SoftwareProperties.py (+1/-2)
softwareproperties/qt/SoftwarePropertiesQt.py (+29/-29)
Text conflict in debian/changelog
To merge this branch: bzr merge lp:~hmollercl/software-properties/software-properties
Reviewer Review Type Date Requested Status
ԜаӀtеr Ⅼарсһуnѕkі Pending
Simon Quigley Pending
Review via email: mp+363293@code.launchpad.net

Commit message

Change debconf frontend of software-properties-qt from gnome to kde.
Added debconf-kde-helper as requiered dependency for software-properties-qt.

Description of the change

Gnome debconf frontend doesn't work easily on Qt DE. On the other hand kde debconf frontend works perfect when debconf-kde-helper is in place

To post a comment you must log in.
Revision history for this message
Simon Quigley (tsimonq2) wrote :

Could you please add a changelog entry?

1069. By Hans P. Möller

added changes to changelog

1070. By Hans P. Möller

added/changed desktop entries for LXQt

1071. By Hans P. Möller

Solve bug 1801439

1072. By Hans P. Möller

fixing typos

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/designer/main.ui'
--- data/designer/main.ui 2018-12-29 02:11:03 +0000
+++ data/designer/main.ui 2019-02-24 16:52:14 +0000
@@ -558,7 +558,7 @@
558 <x>0</x>558 <x>0</x>
559 <y>0</y>559 <y>0</y>
560 <width>670</width>560 <width>670</width>
561 <height>244</height>561 <height>255</height>
562 </rect>562 </rect>
563 </property>563 </property>
564 <layout class="QVBoxLayout" name="verticalLayout_3">564 <layout class="QVBoxLayout" name="verticalLayout_3">
@@ -611,7 +611,7 @@
611 <item>611 <item>
612 <widget class="QLabel" name="label_driver_action">612 <widget class="QLabel" name="label_driver_action">
613 <property name="text">613 <property name="text">
614 <string>No propietary drivers are in use.</string>614 <string>No proprietary drivers are in use.</string>
615 </property>615 </property>
616 </widget>616 </widget>
617 </item>617 </item>
618618
=== added file 'data/software-properties-drivers-qt.desktop.in'
--- data/software-properties-drivers-qt.desktop.in 1970-01-01 00:00:00 +0000
+++ data/software-properties-drivers-qt.desktop.in 2019-02-24 16:52:14 +0000
@@ -0,0 +1,13 @@
1[Desktop Entry]
2Keywords=Drivers;
3Exec=lxqt-sudo /usr/bin/software-properties-qt --open-tab=4
4Icon=jockey
5Terminal=false
6Type=Application
7OnlyShowIn=LXQt;
8Categories=Settings;HardwareSettings;System;
9X-AppStream-Ignore=true
10Name=Additional Drivers
11Name[es]=Controladores adicionales
12Comment=Configure third-party and proprietary drivers
13Comment[es]=Configure controladores privativos y de terceros
014
=== modified file 'data/software-properties-qt.desktop.in'
--- data/software-properties-qt.desktop.in 2018-07-14 10:11:32 +0000
+++ data/software-properties-qt.desktop.in 2019-02-24 16:52:14 +0000
@@ -2,9 +2,9 @@
2_Name=Software Sources2_Name=Software Sources
3_GenericName=Software Sources3_GenericName=Software Sources
4_Comment=Configure the sources for installable software and updates4_Comment=Configure the sources for installable software and updates
5Exec=software-properties-qt5Exec=lxqt-sudo software-properties-qt
6Icon=applications-other6Icon=applications-other
7NoDisplay=true7OnlyShowIn=LXQt;
8Terminal=false8Terminal=false
9X-MultipleArgs=false9X-MultipleArgs=false
10Type=Application10Type=Application
1111
=== modified file 'debian/changelog'
--- debian/changelog 2019-02-19 09:39:44 +0000
+++ debian/changelog 2019-02-24 16:52:14 +0000
@@ -1,3 +1,4 @@
1<<<<<<< TREE
1software-properties (0.97.6) disco; urgency=medium2software-properties (0.97.6) disco; urgency=medium
23
3 * debian/control:4 * debian/control:
@@ -31,6 +32,19 @@
3132
32 -- Andrea Azzarone <andrea.azzarone@canonical.com> Tue, 29 Jan 2019 18:29:18 +000033 -- Andrea Azzarone <andrea.azzarone@canonical.com> Tue, 29 Jan 2019 18:29:18 +0000
3334
35=======
36software-properties (0.97.3) disco; urgency=medium
37
38 [ Hans P. Möller ]
39 * Change debconf frontend to kde for software-properties-qt.
40 * Change Qt Desktop Entry to be shown in LXQt.
41 * Add Drivers Qt Desktop Entry to be shown in LXQt.
42 * Solve Bug 1801439
43 * Fixed typos in Qt Additional Driver tab.
44
45 -- Hans P. Moller <hmoller@gmail.com> Thu, 21 Feb 2019 22:49:39 -0300
46
47>>>>>>> MERGE-SOURCE
34software-properties (0.97.2) disco; urgency=medium48software-properties (0.97.2) disco; urgency=medium
3549
36 * Install python3-aptdaemon with software-properties-qt.50 * Install python3-aptdaemon with software-properties-qt.
3751
=== modified file 'debian/control'
--- debian/control 2019-02-19 09:39:32 +0000
+++ debian/control 2019-02-24 16:52:14 +0000
@@ -83,7 +83,7 @@
83Architecture: all83Architecture: all
84Depends: ${python3:Depends}, ${misc:Depends}, python3, python3-pyqt5,84Depends: ${python3:Depends}, ${misc:Depends}, python3, python3-pyqt5,
85 python3-software-properties (= ${binary:Version}), python3-aptdaemon,85 python3-software-properties (= ${binary:Version}), python3-aptdaemon,
86 qapt-batch, software-properties-common86 qapt-batch, software-properties-common, debconf-kde-helper
87Conflicts: software-properties-kde (<< 0.96.25)87Conflicts: software-properties-kde (<< 0.96.25)
88Replaces: software-properties-kde (<< 0.96.25)88Replaces: software-properties-kde (<< 0.96.25)
89Description: manage the repositories that you install software from (Qt)89Description: manage the repositories that you install software from (Qt)
9090
=== modified file 'softwareproperties/SoftwareProperties.py'
--- softwareproperties/SoftwareProperties.py 2019-01-29 18:24:52 +0000
+++ softwareproperties/SoftwareProperties.py 2019-02-24 16:52:14 +0000
@@ -694,8 +694,7 @@
694 keyp = "%s/%s.key" % (self.CHANNEL_PATH, channel)694 keyp = "%s/%s.key" % (self.CHANNEL_PATH, channel)
695 self.add_key(keyp)695 self.add_key(keyp)
696696
697 cdata = (shortcut.add_key, {'keyserver': (self.options and697 cdata = (shortcut.add_key, {'keyserver': (self.options)})
698 self.options.keyserver)})
699 def addkey_func():698 def addkey_func():
700 func, kwargs = cdata699 func, kwargs = cdata
701 msg = "Added key."700 msg = "Added key."
702701
=== modified file 'softwareproperties/qt/SoftwarePropertiesQt.py'
--- softwareproperties/qt/SoftwarePropertiesQt.py 2019-02-09 23:11:11 +0000
+++ softwareproperties/qt/SoftwarePropertiesQt.py 2019-02-24 16:52:14 +0000
@@ -125,7 +125,7 @@
125 self.show_cdrom_sources()125 self.show_cdrom_sources()
126 # Setup and show the Additonal Drivers tab126 # Setup and show the Additonal Drivers tab
127 self.init_drivers()127 self.init_drivers()
128 128
129 # Connect to switch-page before setting initial tab. Otherwise the129 # Connect to switch-page before setting initial tab. Otherwise the
130 # first switch goes unnoticed.130 # first switch goes unnoticed.
131 self.userinterface.tabWidget.currentChanged.connect(self.tab_switched)131 self.userinterface.tabWidget.currentChanged.connect(self.tab_switched)
@@ -145,7 +145,7 @@
145 self.userinterface.treeview_sources.itemClicked.connect(self.on_treeview_sources_cursor_changed)145 self.userinterface.treeview_sources.itemClicked.connect(self.on_treeview_sources_cursor_changed)
146 self.userinterface.treeview_cdroms.itemChanged.connect(self.on_cdrom_source_toggled)146 self.userinterface.treeview_cdroms.itemChanged.connect(self.on_cdrom_source_toggled)
147 self.userinterface.treeview2.itemClicked.connect(self.on_treeview_keys_cursor_changed)147 self.userinterface.treeview2.itemClicked.connect(self.on_treeview_keys_cursor_changed)
148 148
149 self.button_close = self.userinterface.buttonBox.button(QDialogButtonBox.Close)149 self.button_close = self.userinterface.buttonBox.button(QDialogButtonBox.Close)
150 self.button_close.setIcon(QIcon.fromTheme("dialog-close"))150 self.button_close.setIcon(QIcon.fromTheme("dialog-close"))
151 self.button_revert = self.userinterface.buttonBox.button(QDialogButtonBox.Reset)151 self.button_revert = self.userinterface.buttonBox.button(QDialogButtonBox.Reset)
@@ -154,27 +154,27 @@
154154
155 self.init_distro()155 self.init_distro()
156 self.show_distro()156 self.show_distro()
157 157
158 self.apt_client = client.AptClient()158 self.apt_client = client.AptClient()
159 159
160 if options and options.open_tab:160 if options and options.open_tab:
161 self.userinterface.tabWidget.setCurrentIndex(int(options.open_tab))161 self.userinterface.tabWidget.setCurrentIndex(int(options.open_tab))
162 162
163 def tab_switched(self):163 def tab_switched(self):
164 # On the additional drivers page, don't show the backend revert button.164 # On the additional drivers page, don't show the backend revert button.
165 if self.userinterface.tabWidget.currentIndex() == 4: #vbox_drivers is 4165 if self.userinterface.tabWidget.currentIndex() == 4: #vbox_drivers is 4
166 self.button_revert.setVisible(False)166 self.button_revert.setVisible(False)
167 if not self.detect_called:167 if not self.detect_called:
168 #WARNING detect_drivers() runs in separate thread 168 #WARNING detect_drivers() runs in separate thread
169 #in DetectDriverThread class so GUI won't freeze169 #in DetectDriverThread class so GUI won't freeze
170 #after finish show_drivers() has to run in main thread because it updates the GUI170 #after finish show_drivers() has to run in main thread because it updates the GUI
171 self.detect_driver_thread = DetectDriverThread(self)171 self.detect_driver_thread = DetectDriverThread(self)
172 self.detect_driver_thread.finished.connect(self.show_drivers)172 self.detect_driver_thread.finished.connect(self.show_drivers)
173 self.detect_driver_thread.start()173 self.detect_driver_thread.start()
174 174
175 else:175 else:
176 self.button_revert.setVisible(True)176 self.button_revert.setVisible(True)
177 177
178 def init_popcon(self):178 def init_popcon(self):
179 """ If popcon is enabled show the statistics tab and an explanation179 """ If popcon is enabled show the statistics tab and an explanation
180 corresponding to the used distro """180 corresponding to the used distro """
@@ -186,7 +186,7 @@
186 self.userinterface.checkbutton_popcon.setChecked(is_helpful)186 self.userinterface.checkbutton_popcon.setChecked(is_helpful)
187 else:187 else:
188 self.userinterface.tabWidget.removeTab(5)188 self.userinterface.tabWidget.removeTab(5)
189 189
190 def init_server_chooser(self):190 def init_server_chooser(self):
191 """ Set up the widgets that allow to choose an alternate download site """191 """ Set up the widgets that allow to choose an alternate download site """
192 # nothing to do here, set up signal in show_distro()192 # nothing to do here, set up signal in show_distro()
@@ -855,7 +855,7 @@
855 self.transaction.connect('cancellable-changed', self.on_driver_changes_cancellable_changed)855 self.transaction.connect('cancellable-changed', self.on_driver_changes_cancellable_changed)
856 self.transaction.connect('finished', self.on_driver_changes_finish)856 self.transaction.connect('finished', self.on_driver_changes_finish)
857 self.transaction.connect('error', self.on_driver_changes_error)857 self.transaction.connect('error', self.on_driver_changes_error)
858 self.transaction.set_debconf_frontend("gnome")858 self.transaction.set_debconf_frontend("kde")
859 self.transaction.run()859 self.transaction.run()
860 self.button_driver_revert.setEnabled(False)860 self.button_driver_revert.setEnabled(False)
861 self.button_driver_apply.setEnabled(False)861 self.button_driver_apply.setEnabled(False)
@@ -902,26 +902,26 @@
902 def clear_changes(self):902 def clear_changes(self):
903 self.orig_selection = {}903 self.orig_selection = {}
904 self.driver_changes = []904 self.driver_changes = []
905 905
906 def init_drivers(self):906 def init_drivers(self):
907 """Additional Drivers tab"""907 """Additional Drivers tab"""
908 self.button_driver_revert = QPushButton("Revert")908 self.button_driver_revert = QPushButton("Revert")
909 self.button_driver_apply = QPushButton("Apply Changes")909 self.button_driver_apply = QPushButton("Apply Changes")
910 self.button_driver_cancel = QPushButton("Cancel")910 self.button_driver_cancel = QPushButton("Cancel")
911 self.button_driver_restart = QPushButton("Restart...")911 self.button_driver_restart = QPushButton("Restart...")
912 912
913 self.button_driver_revert.clicked.connect(self.on_driver_changes_revert)913 self.button_driver_revert.clicked.connect(self.on_driver_changes_revert)
914 self.button_driver_apply.clicked.connect(self.on_driver_changes_apply)914 self.button_driver_apply.clicked.connect(self.on_driver_changes_apply)
915 self.button_driver_cancel.clicked.connect(self.on_driver_changes_cancel)915 self.button_driver_cancel.clicked.connect(self.on_driver_changes_cancel)
916 self.button_driver_restart.clicked.connect(self.on_driver_restart_clicked)916 self.button_driver_restart.clicked.connect(self.on_driver_restart_clicked)
917 917
918 self.button_driver_revert.setEnabled(False)918 self.button_driver_revert.setEnabled(False)
919 self.button_driver_revert.setVisible(True)919 self.button_driver_revert.setVisible(True)
920 self.button_driver_apply.setEnabled(False)920 self.button_driver_apply.setEnabled(False)
921 self.button_driver_apply.setVisible(True)921 self.button_driver_apply.setVisible(True)
922 self.button_driver_cancel.setVisible(False)922 self.button_driver_cancel.setVisible(False)
923 self.button_driver_restart.setVisible(False)923 self.button_driver_restart.setVisible(False)
924 924
925 #self.userinterface.box_driver_action.addWidget(self.userinterface.label_driver_action)925 #self.userinterface.box_driver_action.addWidget(self.userinterface.label_driver_action)
926 self.userinterface.box_driver_action.addStretch()926 self.userinterface.box_driver_action.addStretch()
927 self.userinterface.box_driver_action.addWidget(self.button_driver_apply)927 self.userinterface.box_driver_action.addWidget(self.button_driver_apply)
@@ -934,7 +934,7 @@
934 self.userinterface.box_driver_detail.addWidget(self.label_driver_detail)934 self.userinterface.box_driver_detail.addWidget(self.label_driver_detail)
935935
936 self.progress_bar = QProgressBar()936 self.progress_bar = QProgressBar()
937 937
938 self.userinterface.box_driver_action.addWidget(self.progress_bar)938 self.userinterface.box_driver_action.addWidget(self.progress_bar)
939 self.progress_bar.setVisible(False)939 self.progress_bar.setVisible(False)
940940
@@ -950,7 +950,7 @@
950 self.no_drv = []950 self.no_drv = []
951 self.nonfree_drivers = 0951 self.nonfree_drivers = 0
952 self.ui_building = False952 self.ui_building = False
953 953
954 def detect_drivers(self):954 def detect_drivers(self):
955 # WARNING: This is run in a separate thread.955 # WARNING: This is run in a separate thread.
956 self.detect_called = True956 self.detect_called = True
@@ -969,7 +969,7 @@
969 button = self.userinterface.sender()969 button = self.userinterface.sender()
970 #print(modalias)970 #print(modalias)
971 #print(pkg_name)971 #print(pkg_name)
972 972
973 if self.ui_building:973 if self.ui_building:
974 return974 return
975975
@@ -1017,31 +1017,31 @@
1017 the drivers dict is using this form:1017 the drivers dict is using this form:
1018 {"recommended/alternative": {pkg_name: {1018 {"recommended/alternative": {pkg_name: {
1019 'selected': True/False1019 'selected': True/False
1020 'description': 1020 'description':
1021'description'1021'description'
1022 'builtin': True/False1022 'builtin': True/False
1023 }1023 }
1024 }}1024 }}
1025 "manually_installed": {"manual": {'selected': True, 'description': 1025 "manually_installed": {"manual": {'selected': True, 'description':
1026description_string}}1026description_string}}
1027 "no_driver": {"no_driver": {'selected': True/False, 'description': 1027 "no_driver": {"no_driver": {'selected': True/False, 'description':
1028description_string}}1028description_string}}
10291029
1030 Please note that either manually_installed and no_driver are set to 1030 Please note that either manually_installed and no_driver are set to
1031None if not applicable1031None if not applicable
1032 (no_driver isn't present if there are builtins)1032 (no_driver isn't present if there are builtins)
1033 '''1033 '''
10341034
1035 possible_overall_status = {1035 possible_overall_status = {
1036 'recommended': (_("This device is using the recommended driver."), 1036 'recommended': (_("This device is using the recommended driver."),
1037"recommended-driver"),1037"recommended-driver"),
1038 'alternative': (_("This device is using an alternative driver."), 1038 'alternative': (_("This device is using an alternative driver."),
1039"other-driver"),1039"other-driver"),
1040 'manually_installed': (_("This device is using a manually-installed driver."), "other-driver"),1040 'manually_installed': (_("This device is using a manually-installed driver."), "other-driver"),
1041 'no_driver': (_("This device is not working."), "disable-device")1041 'no_driver': (_("This device is not working."), "disable-device")
1042 }1042 }
10431043
1044 returned_drivers = {'recommended': {}, 'alternative': {}, 1044 returned_drivers = {'recommended': {}, 'alternative': {},
1045'manually_installed': {}, 'no_driver': {}}1045'manually_installed': {}, 'no_driver': {}}
1046 have_builtin = False1046 have_builtin = False
1047 one_selected = False1047 one_selected = False
@@ -1091,7 +1091,7 @@
1091 base_string = _("{base_description} ({licence}, tested)")1091 base_string = _("{base_description} ({licence}, tested)")
1092 else:1092 else:
1093 base_string = _("{base_description} ({licence})")1093 base_string = _("{base_description} ({licence})")
1094 description = base_string.format(base_description=description, 1094 description = base_string.format(base_description=description,
1095licence=licence)1095licence=licence)
10961096
1097 selected = False1097 selected = False
@@ -1132,9 +1132,9 @@
1132 return1132 return
1133 else:1133 else:
1134 self.label_driver_detail.hide()1134 self.label_driver_detail.hide()
1135 1135
1136 self.option_group = {}1136 self.option_group = {}
1137 self.radio_button = {} 1137 self.radio_button = {}
1138 self.ui_building = True1138 self.ui_building = True
1139 self.dynamic_device_status = {}1139 self.dynamic_device_status = {}
1140 for device in sorted(self.devices.keys()):1140 for device in sorted(self.devices.keys()):
@@ -1170,7 +1170,7 @@
1170 else:1170 else:
1171 self.option_group[device] = QButtonGroup()1171 self.option_group[device] = QButtonGroup()
1172 self.option_group[device].addButton(self.radio_button)1172 self.option_group[device].addButton(self.radio_button)
1173 1173
1174 device_detail.addWidget(self.radio_button)1174 device_detail.addWidget(self.radio_button)
1175 self.radio_button.setChecked(drivers[section][driver]['selected'])1175 self.radio_button.setChecked(drivers[section][driver]['selected'])
11761176
@@ -1216,7 +1216,7 @@
1216 self.nonfree_drivers = self.nonfree_drivers + 11216 self.nonfree_drivers = self.nonfree_drivers + 1
12171217
1218 if self.nonfree_drivers > 0:1218 if self.nonfree_drivers > 0:
1219 text = "%s proprietary driver in use." % ( self.nonfree_drivers)1219 text = "%s proprietary driver(s) in use." % ( self.nonfree_drivers)
1220 self.userinterface.label_driver_action.setText(text)1220 self.userinterface.label_driver_action.setText(text)
1221 else:1221 else:
1222 self.userinterface.label_driver_action.setText(_("No proprietary drivers are in use."))1222 self.userinterface.label_driver_action.setText(_("No proprietary drivers are in use."))

Subscribers

People subscribed via source and target branches

to status/vote changes: