From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Reuben Thomas Newsgroups: gmane.emacs.bugs Subject: bug#27833: 25.2; Add support for manpath command to woman.el Date: Wed, 26 Jul 2017 15:33:52 +0100 Message-ID: References: <871sp4xhq8.fsf@sc3d.org> <83shhj5l6r.fsf@gnu.org> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="001a113cceea4dabf80555395699" X-Trace: blaine.gmane.org 1501079720 29146 195.159.176.226 (26 Jul 2017 14:35:20 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Wed, 26 Jul 2017 14:35:20 +0000 (UTC) Cc: 27833@debbugs.gnu.org To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Wed Jul 26 16:35:10 2017 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1daNP4-00079d-9L for geb-bug-gnu-emacs@m.gmane.org; Wed, 26 Jul 2017 16:35:10 +0200 Original-Received: from localhost ([::1]:38525 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1daNP9-0002y2-UL for geb-bug-gnu-emacs@m.gmane.org; Wed, 26 Jul 2017 10:35:15 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:34229) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1daNO2-00022H-5p for bug-gnu-emacs@gnu.org; Wed, 26 Jul 2017 10:34:07 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1daNNy-0007hN-6n for bug-gnu-emacs@gnu.org; Wed, 26 Jul 2017 10:34:06 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:54835) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1daNNy-0007hF-4b for bug-gnu-emacs@gnu.org; Wed, 26 Jul 2017 10:34:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1daNNx-0000B2-Vn for bug-gnu-emacs@gnu.org; Wed, 26 Jul 2017 10:34:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Reuben Thomas Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 26 Jul 2017 14:34:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 27833 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 27833-submit@debbugs.gnu.org id=B27833.1501079641673 (code B ref 27833); Wed, 26 Jul 2017 14:34:01 +0000 Original-Received: (at 27833) by debbugs.gnu.org; 26 Jul 2017 14:34:01 +0000 Original-Received: from localhost ([127.0.0.1]:57512 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1daNNw-0000An-M3 for submit@debbugs.gnu.org; Wed, 26 Jul 2017 10:34:00 -0400 Original-Received: from mail-oi0-f43.google.com ([209.85.218.43]:33490) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1daNNv-0000Aa-1V for 27833@debbugs.gnu.org; Wed, 26 Jul 2017 10:33:59 -0400 Original-Received: by mail-oi0-f43.google.com with SMTP id a9so81123761oih.0 for <27833@debbugs.gnu.org>; Wed, 26 Jul 2017 07:33:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sc3d.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=zOmERIXCVTahKYfCWuIVPDWQfauXCwLnxNoVXQTbb68=; b=EmomhzgJPanY+ZTWIzrkHf5yvBF2+kqrT9elkBfg/oJZ8CS3LU7QeBy8sMpN2SUNmC 9MDfruTsfq37mSiwYaHGemPVhCnpCl4HEJryh7uD58ZpXU7qTOjrDmhQ8PZlrvVRCJ2+ eVXTXKC+hyMUraGUSKmvD+lGJJjBAsbhd4jlI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=zOmERIXCVTahKYfCWuIVPDWQfauXCwLnxNoVXQTbb68=; b=cwPYs5g9GCoivXVvfB8GJLdui9FwBXHTFWk1gYmD3+G2wle3+J90ik0+mhqCXU6sYE 9h4L1bnthNx5HlrIwyOM3xKsZcCYdJ9AyTPnqf9VJta0tDvnWyj7OBatUZ9v+3c2+Sin zPi4jz5uagNYZdEFlWy0vXB5lnQm9PP1Tb4yIJMhcgP9M6tB8G2MOCoOAw1RH4UnOTJr X38+djUWLX6PGup546Q6n/NCEBpXm8/5wa/qWjdRnBeU2tpSlWApfI3XHhwKlfG6HNMZ eEMwRagVLw3QUWXrK+3C3KGEBtAbldie0x4w0MClBnrxR7fZK4z3VGy/LbbcgphqXLiM MjXg== X-Gm-Message-State: AIVw112b8KKHx+J0KZVWlI8WIMAfuKkOoRJMOjeWo7Dm0ddslcDfW+5W 7966MrWTLRKuh5LFVb/CEGpv6zGm4Syu X-Received: by 10.202.185.70 with SMTP id j67mr925300oif.138.1501079632971; Wed, 26 Jul 2017 07:33:52 -0700 (PDT) Original-Received: by 10.202.183.87 with HTTP; Wed, 26 Jul 2017 07:33:52 -0700 (PDT) In-Reply-To: <83shhj5l6r.fsf@gnu.org> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:135015 Archived-At: --001a113cceea4dabf80555395699 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On 26 July 2017 at 15:26, Eli Zaretskii wrote: > > From: Reuben Thomas > > Date: Tue, 25 Jul 2017 23:41:03 +0100 > > > > The attached patch against git master adds support for the man-db > > manpath command to woman.el. > > > > For now, this can actually be better than using man, as man -k/apropos > > does not currently support man directories lacking a database or whatis > > file, which is typical for directories derived from PATH. > > Could you please provide a rationale for this addition? > > It sounds strange to me to enhance woman.el, whose main purpose is to > work without ("wo") 'man', including on systems where 'man' is not > easily available, by making it run 'manpath'. If something is wrong > with woman-manpath's built-in database, why cannot we extend it by > adding potential candidates to those already existing there? > =E2=80=8BYou're right about woman being designed to run without man, I hadn= 't thought of that. There's nothing wrong with woman-manpath's built-in database. The idea is simply to use "manpath"'s extra functionality, which looks for man pages in directories formed from the directories on PATH. I guess the logical thing to do would therefore be to replicate this functionality directly, but I don't propose to do it.=E2=80=8B > On top of that, I think we semi-deprecated woman.el because AFAIR it > cannot support newer roff features which are abundant in recent man > pages. So I wonder what kind of use case do you have that on the one > hand needs woman.el, and OTOH finds its manpath emulation lacking. > =E2=80=8B =E2=80=8BI explained that: man -k does not support man directories lacking= a database. I've fixed that for my own purposes by running "mandb" as a per-user cronjob, so I suggest you can ignore this patch (which I agree is ill-conceived), and close the bug report. Thanks as always for your penetrating analysis. --=20 https://rrt.sc3d.org --001a113cceea4dabf80555395699 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
On = 26 July 2017 at 15:26, Eli Zaretskii <eliz@gnu.org> wrote:
> From: Reuben Thomas <= rrt@sc3d.org>
> Date: Tue, 25 Jul 2017 23:41:03 +0100
>
> The attached patch against git master adds support for the man-db
> manpath command to woman.el.
>
> For now, this can actually be better than using man, as man -k/apropos=
> does not currently support man directories lacking a database or whati= s
> file, which is typical for directories derived from PATH.

Could you please provide a rationale for this addition?

It sounds strange to me to enhance woman.el, whose main purpose is to
work without ("wo") 'man', including on systems where = 9;man' is not
easily available, by making it run 'manpath'.=C2=A0 If something is= wrong
with woman-manpath's built-in database, why cannot we extend it by
adding potential candidates to those already existing there?

=E2=80=8BYou're right about woman being designed to run without man, = I hadn't thought of that.

There's nothing wrong with woman-manpath's built-in database. = The idea is simply to use "manpath"'s extra functionality, wh= ich looks for man pages in directories formed from the directories on PATH.=

I guess the logical thin= g to do would therefore be to replicate this functionality directly, but I = don't propose to do it.=E2=80=8B
=C2=A0
On top of that, I think we semi-deprecated woman.el be= cause AFAIR it
cannot support newer roff features which are abundant in recent man
pages.=C2=A0 So I wonder what kind of use case do you have that on the one<= br> hand needs woman.el, and OTOH finds its manpath emulation lacking.
= =E2=80=8B
=E2= =80=8BI explained =C2=A0that: man -k does not support man directories lacki= ng a database.
=
I've f= ixed that for my own purposes by running "mandb" as a per-user cr= onjob, so I suggest you can ignore this patch (which I agree is ill-conceiv= ed), and close the bug report.

Thanks as always for your penetrating analysis.

-= -
--001a113cceea4dabf80555395699--