summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabriel F. T. Gomes <gabriel@inconstante.eti.br>2018-11-10 15:31:27 -0200
committerGabriel F. T. Gomes <gabriel@inconstante.eti.br>2018-11-10 16:08:55 -0200
commitf933650579234d1329f3b10de232ce802bbb772b (patch)
tree8453d6c742537ecaf445f6c3d588267b48180d33
parent97c60ae8446c207b6a3a073d3649967e039c161b (diff)
downloadbash-completion-f933650579234d1329f3b10de232ce802bbb772b.tar.gz
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
-rw-r--r--debian/changelog3
-rw-r--r--debian/patches/09-fallbacks-for-man-completion.patch38
-rw-r--r--debian/patches/series1
3 files changed, 3 insertions, 39 deletions
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 <gabriel@inconstante.eti.br>]
+ * Revert `Extra fix for man completion' (from version 1:2.8-2).
+
-- Gabriel F. T. Gomes <gabriel@inconstante.eti.br> 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 <pabs@debian.org>
-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 <pawel.veselov@gmail.com>
- 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