Merge ~nteodosio/update-manager:bionic-ua into update-manager:ubuntu/bionic

Proposed by Nathan Teodosio
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 55a1fb6a73727f0ea8d4c34012d7817f1065e636
Merged at revision: 55a1fb6a73727f0ea8d4c34012d7817f1065e636
Proposed branch: ~nteodosio/update-manager:bionic-ua
Merge into: update-manager:ubuntu/bionic
Diff against target: 50 lines (+31/-0)
2 files modified
data/gtkbuilder/UpdateManager.ui (+24/-0)
debian/changelog (+7/-0)
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+457180@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/data/gtkbuilder/UpdateManager.ui b/data/gtkbuilder/UpdateManager.ui
2index d5321a2..7a53445 100644
3--- a/data/gtkbuilder/UpdateManager.ui
4+++ b/data/gtkbuilder/UpdateManager.ui
5@@ -5,6 +5,30 @@
6 <property name="visible">True</property>
7 <property name="can_focus">False</property>
8 <property name="spacing">12</property>
9+ <!--
10+ https://launchpad.net/bugs/2045918
11+ Old widgets from 18.04.11 need to be kept, even though invisible, because
12+ otherwise updating Update Manager itself via Update Manager will crash when
13+ Update Manager recalculates itself. That is because the old Update Manager
14+ file is still the being executed, whereas this ui file will be the new one.
15+ -->
16+ <child>
17+ <object class="GtkNotebook" id="notebook_details">
18+ <property name="visible">False</property>
19+ </object>
20+ </child>
21+ <child>
22+ <object class="GtkExpander" id="expander_desc">
23+ <property name="visible">False</property>
24+ </object>
25+ </child>
26+ <child>
27+ <object class="GtkTextView" id="textview_descr">
28+ <property name="visible">False</property>
29+ </object>
30+ </child>
31+ <!-- End of old widgets -->
32+
33 <child>
34 <object class="GtkExpander" id="expander_news">
35 <property name="visible">False</property>
36diff --git a/debian/changelog b/debian/changelog
37index 9e8c039..9e83e4e 100644
38--- a/debian/changelog
39+++ b/debian/changelog
40@@ -1,3 +1,10 @@
41+update-manager (1:18.04.11.22) bionic; urgency=medium
42+
43+ * Add back removed widgets to UI file to fix crash when updating
44+ Upgrade Manager from inside itself (LP: #2045918).
45+
46+ -- Nathan Pratta Teodosio <nathan.teodosio@canonical.com> Fri, 08 Dec 2023 15:39:44 +0100
47+
48 update-manager (1:18.04.11.21) bionic; urgency=medium
49
50 * Fix incorrect available version for Ubuntu Pro updates in unattached case

Subscribers

People subscribed via source and target branches