From: Yuri Pankov <yuri.pankov@gmail.com> To: Ingo Schwarze <schwarze@usta.de> Cc: discuss@mdocml.bsd.lv Subject: Re: making all warnings fatal again? Date: Tue, 10 Jan 2017 01:10:37 +0300 [thread overview] Message-ID: <c9c911fa-d32c-557e-4ba2-4d2c01c207d7@gmail.com> (raw) In-Reply-To: <20170109024555.GF4271@athene.usta.de> On Mon, 9 Jan 2017 03:45:55 +0100, Ingo Schwarze wrote: > Hi, > > Yuri Pankov wrote on Sat, Jan 07, 2017 at 05:33:26AM +0300: > >> Switching from 1.13.3 to 1.13.4 made mandoc -Tlint exit 0 >> instead of 2 for *some* of the warnings it reports. >> Is there a way to make all warnings "fatal" again? > > Ouch. That is an evil regression that i missed. > Thank you very much for reporting it! > >> For example (not limited to this one, of course): >> >> 1.13.3: >> >> $ mandoc -Tlint foo.1; echo $? >> mandoc: foo.1:2:5: WARNING: lower case character in document title: Dt foo >> 2 >> >> 1.13.4: >> >> $ mandoc -Tlint foo.1; echo $? >> mandoc: foo.1:2:5: WARNING: lower case character in document title: Dt foo >> 0 > > True, it got broken. Failures in the checks that were moved > to mdoc_validate() and man_validate() got ignored in the mandoc(1) > EXIT STATUS, which was wrong. > > I just fixed it in bsd.lv -current and in OpenBSD with the commit > below. > > I hope that it applies to the illumos tree. It does apply to the > vanilla 1.13.4 release version for me, and it seems to work there, > too, and not just in HEAD. Yes, I've picked up the changes from VERSION_1_13, everything seems to be correct now. Thanks for fixing it! > When 1.13.5 and 1.14 get released, this fix wil be contained, > but there is no release schedule yet. > >> P.S.: are the list archives available somewhere? > > Unfortunately, no. The list server does retain copies of the > messages gone through, but the work of making them public was > never done. -- To unsubscribe send an email to discuss+unsubscribe@mdocml.bsd.lv
prev parent reply other threads:[~2017-01-09 22:10 UTC|newest] Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-01-07 2:33 Yuri Pankov 2017-01-09 2:45 ` Ingo Schwarze 2017-01-09 22:10 ` Yuri Pankov [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=c9c911fa-d32c-557e-4ba2-4d2c01c207d7@gmail.com \ --to=yuri.pankov@gmail.com \ --cc=discuss@mdocml.bsd.lv \ --cc=schwarze@usta.de \ --subject='Re: making all warnings fatal again?' \ /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
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).