On Fri, Sep 18, Andrew Shadura wrote:
> Karl E. Jorgensen wrote:
>> ''You should include "event" here, but you must not do so.'' ????
>> Huh??  To include or to not include.... I'm confused, and I think most
>> other readers would be too :-)
> 
> I'm not sure I understand what exactly is confusing you :)

I'm not the bug submitter, but I see what's confusing him.  Just as he
says, the manual page for brightd has some rather confusing text in the
description of the -e option:

# -e n
#
#  Filter used event sources by POSIX extended regexp n (for example,
#  use "i8042.+event" on intel platforms to avoid having HDAPS taken
#  into account) You should include "event" here, but you must not do
#  so.

Look at that last sentence.
#  You should include "event" here, but you must not do so.

Grammatically, that clause "you must not do so" can be interpreted
only one way: it's saying

#  you SHOULD: include "event" here, but
#  you MUST:   not include "event" here.

In other words, this action is both requested and prohibited.

I suspect that the writer of the manual got confused about the correct
way to negate a "must", and intended it to mean:

#  You should probably include "event" here, but it is not mandatory.

The secondary problem is that it's completely unclear how I would
"include 'event' here" - does "event" mean the literal string 'event'
or the name of an ACPI event type or what?

The whole man page is full of grammatical problems, but the rest of
them are less serious than this.
-- 
JBR     with qualifications in linguistics, experience as a Debian
        sysadmin, and probably no clue about this particular package

Reply via email to