Merge ~3v1n0/bamf:ubuntu/focal into bamf:ubuntu/focal

Proposed by Marco Trevisan (Treviño)
Status: Needs review
Proposed branch: ~3v1n0/bamf:ubuntu/focal
Merge into: bamf:ubuntu/focal
Diff against target: 98 lines (+13/-51)
3 files modified
debian/changelog (+11/-0)
dev/null (+0/-49)
src/bamf-view.c (+2/-2)
Reviewer Review Type Date Requested Status
Unity Team Pending
Review via email: mp+398002@code.launchpad.net
To post a comment you must log in.

Unmerged commits

07e11b9... by Marco Trevisan (Treviño)

Update changelog

f0ddb87... by Corentin Noël

bamf-view: avoid overriding the `timeout` variables after setting them

For instance, calling `active-changed` might reset the active_changed_idle variable
and we are replacing it by a zero just after.

LP: #1915529

c44386e... by Marco Trevisan (Treviño)

debian/patches: Remove already applied patches

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 0b2d73b..161e179 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,14 @@
6+bamf (0.5.3+18.04.20180207.2-0ubuntu3) UNRELEASED; urgency=medium
7+
8+ [ Marco Trevisan (Treviño) ]
9+ * debian/patches: Remove already applied patches
10+
11+ [ Corentin Noël ]
12+ * bamf-view: avoid overriding the `timeout` variables after setting them
13+ (LP: #1915529)
14+
15+ -- Marco Trevisan (Treviño) <marco@ubuntu.com> Fri, 12 Feb 2021 16:55:57 +0100
16+
17 bamf (0.5.3+18.04.20180207.2-0ubuntu2) focal; urgency=medium
18
19 * Cherry-pick from Debian, to fix FTBFS without python-lxml:
20diff --git a/debian/patches/1004_py2to3_gtester2xunit.patch b/debian/patches/1004_py2to3_gtester2xunit.patch
21deleted file mode 100644
22index bd8af9b..0000000
23--- a/debian/patches/1004_py2to3_gtester2xunit.patch
24+++ /dev/null
25@@ -1,49 +0,0 @@
26-Description: Run gtester2xunit.py with Python3, port libxml2/libxslt to lxml module..
27-Author: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
28-Origin: https://salsa.debian.org/debian-mate-team/bamf/-/commit/4c420fa6a035680cd071684c85b3a542bdd6e205
29-
30---- a/tests/gtester2xunit.py
31-+++ b/tests/gtester2xunit.py
32-@@ -1,18 +1,17 @@
33--#! /usr/bin/python
34-+#! /usr/bin/python3
35- from argparse import ArgumentParser
36--import libxslt
37--import libxml2
38-+from lxml import etree
39- import sys
40- import os
41-
42- XSL_TRANSFORM='/usr/share/gtester2xunit/gtester.xsl'
43-
44- def transform_file(input_filename, output_filename, xsl_file):
45-- gtester = libxml2.parseFile(xsl_file)
46-- style = libxslt.parseStylesheetDoc(gtester)
47-- doc = libxml2.parseFile(input_filename)
48-- result = style.applyStylesheet(doc, None)
49-- result.saveFormatFile(filename=output_filename, format=True)
50-+ gtester = etree.parse(xsl_file)
51-+ transform = etree.XSLT(gtester)
52-+ doc = etree.parse(input_filename)
53-+ result = transform(doc)
54-+ result.write(output_filename)
55-
56-
57- def get_output_filename(input_filename):
58---- a/configure.ac
59-+++ b/configure.ac
60-@@ -109,11 +109,11 @@
61- # gtester2xunit checks #
62- ###########################
63-
64--AC_PATH_PROG([PYTHON],[python])
65-+AC_PATH_PROG([PYTHON],[python3])
66- AC_MSG_CHECKING(for gtester2xunit dependencies)
67--if !($PYTHON -c "import libxslt, libxml2" 2> /dev/null); then
68-+if !($PYTHON -c "from lxml import etree" 2> /dev/null); then
69- AC_MSG_RESULT([no])
70-- AC_MSG_ERROR([You need to install python-libxslt1 and python-libxml2]);
71-+ AC_MSG_ERROR([You need to install python3-lxml]);
72- fi
73- AC_MSG_RESULT([yes])
74-
75diff --git a/src/bamf-view.c b/src/bamf-view.c
76index 89faec1..08f06bd 100644
77--- a/src/bamf-view.c
78+++ b/src/bamf-view.c
79@@ -92,8 +92,8 @@ on_active_changed_idle (gpointer data)
80 BamfView *self = BAMF_VIEW (data);
81 gboolean active = bamf_view_is_active (self);
82
83- g_signal_emit_by_name (self, "active-changed", active);
84 self->priv->active_changed_idle = 0;
85+ g_signal_emit_by_name (self, "active-changed", active);
86
87 return FALSE;
88 }
89@@ -157,8 +157,8 @@ on_starting_timeout (gpointer data)
90 {
91 BamfView *view = data;
92
93- bamf_view_set_starting (view, NULL, FALSE);
94 view->priv->starting_timeout = 0;
95+ bamf_view_set_starting (view, NULL, FALSE);
96
97 return FALSE;
98 }

Subscribers

People subscribed via source and target branches