Merge lp:~javier-lopez/ubuntu/trusty/xz-utils/fix-1248044 into lp:ubuntu/trusty/xz-utils

Proposed by Javier López
Status: Merged
Merge reported by: Martin Pitt
Merged at revision: not available
Proposed branch: lp:~javier-lopez/ubuntu/trusty/xz-utils/fix-1248044
Merge into: lp:ubuntu/trusty/xz-utils
Diff against target: 42 lines (+22/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/error-parsing-less.patch (+14/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~javier-lopez/ubuntu/trusty/xz-utils/fix-1248044
Reviewer Review Type Date Requested Status
Martin Pitt Approve
Review via email: mp+205494@code.launchpad.net

Description of the change

It has been two years since this bug with a solution was reported, it's already been fixed in Upstream, currently waiting for Debian to packaging a newer release version. In the meantime and if it doesn't get repackaged on time, I though it could be at least been fixed in Ubuntu.

The above change parse correctly newer less output version, and uses the original patch from Jörg-Volker Peetz.

To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks! Uploaded with a DEP-3 patch header.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-01-03 17:21:16 +0000
3+++ debian/changelog 2014-02-09 07:29:17 +0000
4@@ -1,3 +1,10 @@
5+xz-utils (5.1.1alpha+20120614-2ubuntu2) trusty; urgency=low
6+
7+ * debian/patches/error-parsing-less.patch. LP: #1248044
8+ - parse correctly recent versions of less output
9+
10+ -- Javier P.L. <chilicuil@ubuntu.com> Sun, 09 Feb 2014 01:21:14 -0600
11+
12 xz-utils (5.1.1alpha+20120614-2ubuntu1) raring; urgency=low
13
14 * Remove build-dependency on perl; we use no non-trivial Perl modules so
15
16=== added file 'debian/patches/error-parsing-less.patch'
17--- debian/patches/error-parsing-less.patch 1970-01-01 00:00:00 +0000
18+++ debian/patches/error-parsing-less.patch 2014-02-09 07:29:17 +0000
19@@ -0,0 +1,14 @@
20+## Description: add some description
21+## Origin/Author: add some origin or author
22+## Bug: bug URL
23+--- a/src/scripts/xzless.in
24++++ b/src/scripts/xzless.in
25+@@ -46,7 +46,7 @@ if test "${LESSMETACHARS+set}" != set; t
26+ LESSMETACHARS="$space$tab$nl'"';*?"()<>[|&^`#\$%=~'
27+ fi
28+
29+-if test "$(less -V | { read ver && echo ${ver#less }; })" -ge 429; then
30++if test "$(less -V | { read ver && ver=${ver#less } && echo ${ver%% *}; })" -ge 429; then
31+ # less 429 or later: LESSOPEN pipe will be used on
32+ # standard input if $LESSOPEN begins with |-.
33+ LESSOPEN="|-$xz -cdfq -- %s"
34
35=== modified file 'debian/patches/series'
36--- debian/patches/series 2012-09-10 14:35:33 +0000
37+++ debian/patches/series 2014-02-09 07:29:17 +0000
38@@ -6,3 +6,4 @@
39 man-xz-lvv-minver
40 xz-lvv-empty-block-minver
41 decoder-check-first-0x00
42+error-parsing-less.patch

Subscribers

People subscribed via source and target branches

to all changes: