From f933650579234d1329f3b10de232ce802bbb772b Mon Sep 17 00:00:00 2001 From: "Gabriel F. T. Gomes" Date: Sat, 10 Nov 2018 15:31:27 -0200 Subject: Revert "Extra fix for man completion" This reverts commit c4ea37493f4c7c287ea0b0048bb58635f855b7bf. The discussion upstream [1] exposed some potential, undesired side-effects that this fix could cause, hence the reversion. [1] https://github.com/scop/bash-completion/pull/252#issuecomment-435928977 --- debian/changelog | 3 ++ .../patches/09-fallbacks-for-man-completion.patch | 38 ---------------------- debian/patches/series | 1 - 3 files changed, 3 insertions(+), 39 deletions(-) delete mode 100644 debian/patches/09-fallbacks-for-man-completion.patch diff --git a/debian/changelog b/debian/changelog index dc154209..f7de9575 100644 --- a/debian/changelog +++ b/debian/changelog @@ -4,6 +4,9 @@ bash-completion (1:2.8-3) UNRELEASED; urgency=medium * Fix extremely annoying and long-standing breakage of completion within POSIX command substitution $(). (Closes: #742466) + [Gabriel F. T. Gomes ] + * Revert `Extra fix for man completion' (from version 1:2.8-2). + -- Gabriel F. T. Gomes Sun, 04 Nov 2018 18:48:03 -0200 bash-completion (1:2.8-2) unstable; urgency=low diff --git a/debian/patches/09-fallbacks-for-man-completion.patch b/debian/patches/09-fallbacks-for-man-completion.patch deleted file mode 100644 index 023ea903..00000000 --- a/debian/patches/09-fallbacks-for-man-completion.patch +++ /dev/null @@ -1,38 +0,0 @@ -From: Paul Wise -Subject: Fallbacks for man completion -Origin: vendor, https://bugs.debian.org/892307#27 -Bug-Debian: https://bugs.debian.org/892307 -Forwarded: yes, https://github.com/scop/bash-completion/pull/252 - -Upstream commit: - - commit e6a471511dfdc230ff3eed65ccba09b6d7d30262 - Author: Pawel - Date: Wed Sep 27 06:34:59 2017 +0200 - - man: Don't use $MANPATH directly (#161) - -fixes the behavior of man completion when MANPATH is set, however, it -misses the following: - - * falling back on `man --path` when the man implementation does not - have support for `manpath` and `man -w` - * falling back on $MANPATH when the man implementation does not have - support for printing the manual pages paths - -This patch addresses them. - -diff --git a/completions/man b/completions/man -index 0668b8ee..d0068fc7 100644 ---- a/completions/man -+++ b/completions/man -@@ -51,7 +51,8 @@ _man() - return - fi - -- local manpath=$( manpath 2>/dev/null || command man -w 2>/dev/null ) -+ local manpath=$( manpath 2>/dev/null || command man -w 2>/dev/null || command man --path 2>/dev/null ) -+ [[ -z $manpath ]] && manpath=$MANPATH - [[ -z $manpath ]] && manpath="/usr/share/man:/usr/local/share/man" - - # determine manual section to search diff --git a/debian/patches/series b/debian/patches/series index a183936f..cc09e7f1 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -7,5 +7,4 @@ 06-xpdf_support_compressed_pdf.patch 07-dpkg_support_raw-extract_vextract.patch 08-completion-for-cvs-log.patch -09-fallbacks-for-man-completion.patch 99-use-install-data-hook.patch -- cgit v1.2.1