GNU bug report logs - #27833
25.2; Add support for manpath command to woman.el

Previous Next

Package: emacs;

Reported by: Reuben Thomas <rrt <at> sc3d.org>

Date: Tue, 25 Jul 2017 22:42:02 UTC

Severity: wishlist

Found in version 25.2

Done: Stefan Kangas <stefan <at> marxist.se>

Bug is archived. No further changes may be made.

Full log

Message #11 received at 27833 <at> debbugs.gnu.org (full text, mbox):

From: Reuben Thomas <rrt <at> sc3d.org>
To: Mark Oteiza <mvoteiza <at> udel.edu>
Cc: 27833 <at> debbugs.gnu.org
Subject: Re: bug#27833: 25.2; Add support for manpath command to woman.el
Date: Wed, 26 Jul 2017 09:43:22 +0100

[Message part 1 (text/plain, inline)]
On 26 July 2017 at 00:40, Mark Oteiza <mvoteiza <at> udel.edu> wrote:

>
> Reuben Thomas <rrt <at> sc3d.org> writes:
> > +       (woman-parse-colon-path (condition-case nil
> > +                                   (chomp (with-output-to-string
> > +                                            (with-current-buffer
> > +                                                standard-output
> > +                                              (call-process "manpath"
> nil t nil))))
> > +                                 (error nil))))
> > +      (woman-manpath-add-locales
>
> I suspect your chomp is analogous to string-trim-right in subr-x.el
>

​Thanks! I'd had chomp in my Emacs init for so long I had forgotten it
wasn't part of Emacs, and didn't know about string-trim-right. Updated
patch attached (and that function purged from my init, and my personal uses
of it replaced by string-trim-right).

-- 
https://rrt.sc3d.org <http://rrt.sc3d.org>

[Message part 2 (text/html, inline)]
[0001-Try-to-set-woman-manpath-from-output-of-manpath-comm.patch (text/x-patch, attachment)]
This bug report was last modified 5 years and 315 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.