tech@mandoc.bsd.lv
 help / color / mirror / Atom feed
From: Kristaps Dzonsons <kristaps@bsd.lv>
To: tech@mdocml.bsd.lv
Cc: Ingo Schwarze <schwarze@usta.de>, naddy@openbsd.org
Subject: Re: -Tlint parser errors and warnings to stdout
Date: Sun, 02 Oct 2011 17:12:52 +0200	[thread overview]
Message-ID: <4E887F74.3030808@bsd.lv> (raw)
In-Reply-To: <20110928092146.GA5320@iris.usta.de>

On 09/28/2011 11:21 AM, Ingo Schwarze wrote:
> Hi,
>
> Joerg Sonnenberger wrote on Tue, Sep 27, 2011 at 07:14:27AM +0200:
>> On Tue, Sep 27, 2011 at 01:06:32AM +0200, Kristaps Dzonsons wrote:
>
>>> The -Tlint thing is a different story.  At first I thought (Ingo
>>> will recognise this behaviour) No Way In Hell.  But (1) mdoclint in
>>> regress pushes to stdout; (2) lint itself pushes to stdout; and (3)
>>> there's no real notion of portability/compat so we can do what works
>>> best for us. So basically I'm fine with sticking a dup2 in there if
>>> people think it will save time and effort.
>
>> ...or do some refactoring and replace stderr with errout and just
>> reassign the variable?
>
> Here is a patch doing it in about that way, only slightly different.
> Restore the half-removed curparse argument to mandocmsg(),
> than use that to either select stderr or stdout
> for parser diagnostic output.

Let's hold off on this for a little bit, as there are some elements that 
need thought before going down this road (which I'm ok with, I should 
clarify).  Foremost, since mandoc.3 is built to be a library upstream, 
system errors should be reporting by the same (or parallel) interface as 
mandoc parse errors---NOT using perror or whatnot.  This has bugged me 
for a while, but I've not had the motivation to fix it---let me do so 
before we continue along this road.

Thanks,

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

  reply	other threads:[~2011-10-02 15:12 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CAOS-L3iOOY_2nBA5tdv-xVo92rKK7=spNBBz4Qo__6op+CJdJg@mail.gmail.com>
     [not found] ` <CALi5TckNuUDm8+yb967XpeT7rgAbA9YFO=KOgm0HFx6Dghkp+A@mail.gmail.com>
     [not found]   ` <slrnj7pdak.r8j.stu@naiad.spacehopper.org>
     [not found]     ` <CALi5Tcn4M7kYmcx8tpPnC=TVO3tZ14yayOVhf+jrUXxEKZWVKw@mail.gmail.com>
     [not found]       ` <20110925101243.GC4867@iris.usta.de>
     [not found]         ` <20110925111746.GA14018@lain.home>
     [not found]           ` <20110925180227.GJ4867@iris.usta.de>
     [not found]             ` <20110925182641.GA26028@britannica.bec.de>
     [not found]               ` <4E810578.7040008@bsd.lv>
     [not found]                 ` <20110927051427.GA16670@britannica.bec.de>
2011-09-28  9:21                   ` Ingo Schwarze
2011-10-02 15:12                     ` Kristaps Dzonsons [this message]
2011-10-03 22:26                       ` Ingo Schwarze

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=4E887F74.3030808@bsd.lv \
    --to=kristaps@bsd.lv \
    --cc=naddy@openbsd.org \
    --cc=schwarze@usta.de \
    --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).