all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: David Kastrup <dak@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: Fixes to let woman.el deal with MANPATH_MAP elements
Date: Sat, 24 Mar 2007 18:57:13 +0200	[thread overview]
Message-ID: <uaby2bmvq.fsf@gnu.org> (raw)
In-Reply-To: <85r6rebruy.fsf@lola.goethe.zz> (message from David Kastrup on Sat, 24 Mar 2007 16:09:41 +0100)

> From: David Kastrup <dak@gnu.org>
> Date: Sat, 24 Mar 2007 16:09:41 +0100
> 
> Are there actually other users of "woman"?

Yes, there are.

> While I tested the fixes myself on GNU/Linux, woman.el does a _lot_
> of Windows/Cygwin centric stuff

That's because Windows systems don't come with a `man' command
installed.

> (however wise or not it may be to put such code into woman.el
> instead of providing the general facilities elsewhere).

There's no good general solution to mixing Cygwin programs with a
native Windows build of Emacs.  The author of woman.el wanted the
package to be kinder to such mixing that the rest of Emacs.

> If we don't have active users on this list, I don't see a good way to
> get testing coverage apart from checking it in and having it appear in
> the next pretest.

I'm not against committing this in time for the next pretest, but the
next pretest is still some time away, and you should see a heads-up
several days before that.  If by that time no one hollers, I think you
should go ahead and commit it.

Note that in addition to the Cygwin stuff, woman.el should also be
tested on Windows even without Cygwin.  Could people who don't have
man.exe installed on Windows please try the patched version and see
that the patched functions still work?  Thanks.

  reply	other threads:[~2007-03-24 16:57 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-23 10:29 Fixes to let woman.el deal with MANPATH_MAP elements David Kastrup
2007-03-23 14:16 ` Eli Zaretskii
2007-03-23 18:34   ` David Kastrup
2007-03-23 20:03     ` David Kastrup
2007-03-23 20:12       ` Drew Adams
2007-03-23 20:25         ` David Kastrup
2007-03-23 21:09       ` David Kastrup
2007-03-23 18:00 ` Richard Stallman
2007-03-24 15:09   ` David Kastrup
2007-03-24 16:57     ` Eli Zaretskii [this message]
2007-03-24 17:07       ` David Kastrup
2007-03-24 17:39         ` Eli Zaretskii
2007-03-31 15:25   ` David Kastrup

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=uaby2bmvq.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=dak@gnu.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.