tech@mandoc.bsd.lv
 help / color / mirror / Atom feed
From: Kristaps Dzonsons <kristaps@bsd.lv>
To: tech@mdocml.bsd.lv
Subject: Re: whatis(1)
Date: Mon, 28 Nov 2011 09:42:17 +0100	[thread overview]
Message-ID: <4ED34969.10609@bsd.lv> (raw)
In-Reply-To: <20111128003521.GB15140@iris.usta.de>

On 28/11/2011 01:35, Ingo Schwarze wrote:
> Hi Kristaps,
>
> Kristaps Dzonsons wrote on Sun, Nov 27, 2011 at 07:56:30PM +0100:
>
>> To get the ball rolling, I just checked in whatis.1 and the code
>> modified to use your regexp.  I simplified the call into
>> apropos_db.h with termcomp() instead of exprcomp(), as this mode
>> might be useful for man.cgi too.  The search expression is now case
>> insensitive and ~[[:<:]]term[[:>:]].  (Thanks!)
>
> I have merged this to OpenBSD, with a few tweaks, see below:
>
>   - One real bug: termcomp() returned the wrong end of the singly
>     linked list it built, such that only the last word given on
>     the command line was used.
>
>   - Whatever MacOS may be doing, i tweaked the the regex
>     from "Nm,Nd~" to "Nm~", which is current OpenBSD behaviour.
>     When we replace tools, we should not change behaviour at the
>     same time, but propose changes separately - if we want them.
>     In this case, i'm not even convinced the change makes sense.
>     If you ask "whatis mandoc", you clearly want "mandoc(1)" as
>     an answer; but why should you get "eqn(7), roff(7), tbl(7)"
>     as well?  That's, i think, what plain apropos(1) is for.
>     Maybe that's even a bug in MacOS?
>
>   - I suggest to check only the beginning of the command name,
>     such that you can e.g. install as "whatis.m" for testing
>     purposes.
>
> OK to commit these three tweaks to bsd.lv, too?

Great!  I prefer the `Nm' behaviour too; although I understand why Mac 
does it that way, I'm not convinced I like it yet.  Please update the 
whatis.1 manual, by the way, to note these changes.

Thanks,

Kristaps

--
 To unsubscribe send an email to tech+unsubscribe@mdocml.bsd.lv

      reply	other threads:[~2011-11-28  8:42 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-27 12:16 whatis(1) Kristaps Dzonsons
2011-11-27 18:13 ` whatis(1) Ingo Schwarze
2011-11-27 18:56   ` whatis(1) Kristaps Dzonsons
2011-11-28  0:35     ` whatis(1) Ingo Schwarze
2011-11-28  8:42       ` Kristaps Dzonsons [this message]

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=4ED34969.10609@bsd.lv \
    --to=kristaps@bsd.lv \
    --cc=tech@mdocml.bsd.lv \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).