Merge ~utkarsh/ubuntu/+source/xterm:merge-xterm-impish into ubuntu/+source/xterm:debian/sid

Proposed by Utkarsh Gupta
Status: Merged
Merge reported by: Utkarsh Gupta
Merged at revision: 152cd209e21aa5e80f7910f839bee7db8781db4d
Proposed branch: ~utkarsh/ubuntu/+source/xterm:merge-xterm-impish
Merge into: ubuntu/+source/xterm:debian/sid
Diff against target: 187 lines (+111/-2)
6 files modified
debian/changelog (+82/-0)
debian/control (+2/-1)
debian/gbp.conf (+1/-1)
debian/patches/950_ubuntu_charclass_highlight.diff (+13/-0)
debian/patches/951_uxterm_utf8_title.diff (+11/-0)
debian/patches/series (+2/-0)
Reviewer Review Type Date Requested Status
Gianfranco Costamagna (community) Approve
Utkarsh Gupta Pending
git-ubuntu developers Pending
Review via email: mp+403238@code.launchpad.net

Description of the change

Hello,

This MP merges 366-1 from Debian sid to Impish. PPA could be found at:
https://launchpad.net/~utkarsh/+archive/ubuntu/experimental-dump/

Build is good and autopkgtest passes.

Requesting you to please review and sponsor the upload. TIA! \o/

To post a comment you must log in.
Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

I sponsored after removing the trailing newline on d/changelog and refreshing one patch
Applying patch 950_ubuntu_charclass_highlight.diff
patching file XTerm.ad
Hunk #1 succeeded at 251 (offset 14 lines).

review: Approve
Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Thank you, Gianfranco! :D

Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

This is the change:
diff --git a/debian/changelog b/debian/changelog
index 2d044cb..c6cf7a7 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2108,4 +2108,3 @@ xorg-x11 (6.8.2.dfsg.1-0pre1v1) experimental; urgency=low
     driver.) Thanks to Jesus Climent for spotting this problem.

  -- David Nusinow <email address hidden> Wed, 22 Jun 2005 11:24:38 -0400
-
diff --git a/debian/patches/950_ubuntu_charclass_highlight.diff b/debian/patches/950_ubuntu_charclass_highlight.diff
index 8136fd3..0eb5d17 100644
--- a/debian/patches/950_ubuntu_charclass_highlight.diff
+++ b/debian/patches/950_ubuntu_charclass_highlight.diff
@@ -2,7 +2,7 @@ Index: b/XTerm.ad
 ===================================================================
 --- a/XTerm.ad
 +++ b/XTerm.ad
-@@ -237,7 +237,7 @@
+@@ -251,7 +251,7 @@
  !*faceSize: 8

  ! Here is a pattern that is useful for double-clicking on a URL:

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index 4bf029b..2d044cb 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,24 @@
1xterm (366-1ubuntu1) impish; urgency=medium
2
3 * Merge with Debian unstable. Remaining changes:
4 - Add 950_ubuntu_charclass_highlight.diff:
5 + Enable URL highlighting.
6 - Add 951_uxterm_utf8_title.diff:
7 + Set utf8Titles to true by default when using uxterm, so that it
8 displays utf8 directories in titles properly. May cause issues with
9 apps that use control sequences for updating the xterm titlebar -
10 users should use xterm or set utf8Title to false in this case.
11 - Modify debian/gbp.conf: Set "debian-branch" to "ubuntu"
12 * Dropped changes:
13 - debian/patches/CVE-2021-27135.patch: correct upper-limit for
14 selection buffer, accounting for combining characters in button.c.
15 [Included in 366-1]
16 - debian/patches/CVE-2021-27135-2.patch: check realloc return code,
17 add some casts in button.c.
18 [Included in 366-1]
19
20 -- Utkarsh Gupta <utkarsh@debian.org> Tue, 25 May 2021 11:04:02 +0530
21
1xterm (366-1) unstable; urgency=medium22xterm (366-1) unstable; urgency=medium
223
3 * New upstream release24 * New upstream release
@@ -32,6 +53,38 @@ xterm (362-1) unstable; urgency=medium
3253
33 -- Sven Joachim <svenjoac@gmx.de> Sun, 15 Nov 2020 16:34:36 +010054 -- Sven Joachim <svenjoac@gmx.de> Sun, 15 Nov 2020 16:34:36 +0100
3455
56xterm (361-1ubuntu3) hirsute; urgency=medium
57
58 * SECURITY UPDATE: crash when handling crafted unicode content
59 - debian/patches/CVE-2021-27135-2.patch: check realloc return code,
60 add some casts in button.c.
61 - CVE-2021-27135
62
63 -- Marc Deslauriers <marc.deslauriers@ubuntu.com> Mon, 22 Feb 2021 12:59:13 -0500
64
65xterm (361-1ubuntu2) hirsute; urgency=medium
66
67 * SECURITY UPDATE: crash when handling crafted unicode content
68 - debian/patches/CVE-2021-27135.patch: correct upper-limit for
69 selection buffer, accounting for combining characters in button.c.
70 - CVE-2021-27135
71
72 -- Marc Deslauriers <marc.deslauriers@ubuntu.com> Thu, 11 Feb 2021 09:11:30 -0500
73
74xterm (361-1ubuntu1) hirsute; urgency=low
75
76 * Merge from Debian unstable. Remaining changes:
77 - Add 950_ubuntu_charclass_highlight.diff:
78 + Enable URL highlighting.
79 - Add Add 951_uxterm_utf8_title.diff:
80 + Set utf8Titles to true by default when using uxterm, so that it
81 displays utf8 directories in titles properly. May cause issues with
82 apps that use control sequences for updating the xterm titlebar -
83 users should use xterm or set utf8Title to false in this case.
84 - Modify debian/gbp.conf: Set "debian-branch" to "ubuntu"
85
86 -- Julian Andres Klode <juliank@ubuntu.com> Wed, 11 Nov 2020 17:59:49 +0100
87
35xterm (361-1) unstable; urgency=medium88xterm (361-1) unstable; urgency=medium
3689
37 * New upstream release.90 * New upstream release.
@@ -117,6 +170,20 @@ xterm (354-1) unstable; urgency=medium
117170
118 -- Sven Joachim <svenjoac@gmx.de> Wed, 29 Apr 2020 18:03:39 +0200171 -- Sven Joachim <svenjoac@gmx.de> Wed, 29 Apr 2020 18:03:39 +0200
119172
173xterm (353-1ubuntu1) focal; urgency=medium
174
175 * Sync with Debian, remaining changes:
176 - Add 950_ubuntu_charclass_highlight.diff:
177 + Enable URL highlighting.
178 - Add Add 951_uxterm_utf8_title.diff:
179 + Set utf8Titles to true by default when using uxterm, so that it
180 displays utf8 directories in titles properly. May cause issues with
181 apps that use control sequences for updating the xterm titlebar -
182 users should use xterm or set utf8Title to false in this case.
183 - Modify debian/gbp.conf: Set "debian-branch" to "ubuntu"
184
185 -- Timo Aaltonen <tjaalton@debian.org> Wed, 26 Feb 2020 11:49:57 +0200
186
120xterm (353-1) unstable; urgency=medium187xterm (353-1) unstable; urgency=medium
121188
122 * New upstream release.189 * New upstream release.
@@ -167,6 +234,20 @@ xterm (349-1) unstable; urgency=medium
167234
168 -- Sven Joachim <svenjoac@gmx.de> Mon, 30 Sep 2019 18:25:13 +0200235 -- Sven Joachim <svenjoac@gmx.de> Mon, 30 Sep 2019 18:25:13 +0200
169236
237xterm (348-2ubuntu1) eoan; urgency=medium
238
239 * Sync with Debian, remaining changes:
240 - Add 950_ubuntu_charclass_highlight.diff:
241 + Enable URL highlighting.
242 - Add Add 951_uxterm_utf8_title.diff:
243 + Set utf8Titles to true by default when using uxterm, so that it
244 displays utf8 directories in titles properly. May cause issues with
245 apps that use control sequences for updating the xterm titlebar -
246 users should use xterm or set utf8Title to false in this case.
247 - Modify debian/gbp.conf: Set "debian-branch" to "Ubuntu"
248
249 -- Timo Aaltonen <tjaalton@debian.org> Wed, 18 Sep 2019 15:11:35 +0300
250
170xterm (348-2) unstable; urgency=medium251xterm (348-2) unstable; urgency=medium
171252
172 * Update patch 900_debian_xterm.diff:253 * Update patch 900_debian_xterm.diff:
@@ -2027,3 +2108,4 @@ xorg-x11 (6.8.2.dfsg.1-0pre1v1) experimental; urgency=low
2027 driver.) Thanks to Jesus Climent for spotting this problem.2108 driver.) Thanks to Jesus Climent for spotting this problem.
20282109
2029 -- David Nusinow <dnusinow@debian.org> Wed, 22 Jun 2005 11:24:38 -04002110 -- David Nusinow <dnusinow@debian.org> Wed, 22 Jun 2005 11:24:38 -0400
2111
diff --git a/debian/control b/debian/control
index 0db47e8..d245ea5 100644
--- a/debian/control
+++ b/debian/control
@@ -1,7 +1,8 @@
1Source: xterm1Source: xterm
2Section: x112Section: x11
3Priority: optional3Priority: optional
4Maintainer: Debian X Strike Force <debian-x@lists.debian.org>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Debian X Strike Force <debian-x@lists.debian.org>
5Uploaders: Sven Joachim <svenjoac@gmx.de>6Uploaders: Sven Joachim <svenjoac@gmx.de>
6Standards-Version: 4.5.17Standards-Version: 4.5.1
7Rules-Requires-Root: no8Rules-Requires-Root: no
diff --git a/debian/gbp.conf b/debian/gbp.conf
index bf94336..8d8c4dc 100644
--- a/debian/gbp.conf
+++ b/debian/gbp.conf
@@ -1,6 +1,6 @@
1[DEFAULT]1[DEFAULT]
2pristine-tar = True2pristine-tar = True
3debian-branch = debian-unstable3debian-branch = ubuntu
4upstream-branch = upstream-unstable4upstream-branch = upstream-unstable
5debian-tag = xterm-%(version)s5debian-tag = xterm-%(version)s
6upstream-tag = xterm-%(version)s6upstream-tag = xterm-%(version)s
diff --git a/debian/patches/950_ubuntu_charclass_highlight.diff b/debian/patches/950_ubuntu_charclass_highlight.diff
7new file mode 1006447new file mode 100644
index 0000000..8136fd3
--- /dev/null
+++ b/debian/patches/950_ubuntu_charclass_highlight.diff
@@ -0,0 +1,13 @@
1Index: b/XTerm.ad
2===================================================================
3--- a/XTerm.ad
4+++ b/XTerm.ad
5@@ -237,7 +237,7 @@
6 !*faceSize: 8
7
8 ! Here is a pattern that is useful for double-clicking on a URL:
9-!*charClass: 33:48,35:48,37-38:48,43-47:48,58:48,61:48,63-64:48,95:48,126:48
10+*charClass: 33:48,35:48,37-38:48,43-47:48,58:48,61:48,63-64:48,95:48,126:48
11 !
12 ! Alternatively,
13 !*on2Clicks: regex [[:alpha:]]+://([[:alnum:]!#+,./=?@_~-]|(%[[:xdigit:]][[:xdigit:]]))+
diff --git a/debian/patches/951_uxterm_utf8_title.diff b/debian/patches/951_uxterm_utf8_title.diff
0new file mode 10064414new file mode 100644
index 0000000..2e96921
--- /dev/null
+++ b/debian/patches/951_uxterm_utf8_title.diff
@@ -0,0 +1,11 @@
1diff -Nurp xterm-253/UXTerm.ad xterm-253-working/UXTerm.ad
2--- xterm-253/UXTerm.ad 2006-02-12 17:14:58.000000000 -0800
3+++ xterm-253-working/UXTerm.ad 2010-01-08 13:26:23.000000000 -0800
4@@ -39,6 +39,7 @@
5
6 *fontMenu.Label: Unicode Fonts
7 *VT100.utf8: 1
8+*utf8Title: true
9
10 ! This includes "XTerm-color" which includes "XTerm", which defines fonts.
11 ! Why set them here?
diff --git a/debian/patches/series b/debian/patches/series
index 0a584d1..595515f 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1,5 @@
1900_debian_xterm.diff1900_debian_xterm.diff
2902_windowops.diff2902_windowops.diff
3904_fontops.diff3904_fontops.diff
4950_ubuntu_charclass_highlight.diff
5951_uxterm_utf8_title.diff

Subscribers

People subscribed via source and target branches