All of lore.kernel.org
 help / color / mirror / Atom feed
From: khali@linux-fr.org (Jean Delvare)
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] Sensors-detect with DMI detection
Date: Tue, 27 Feb 2007 08:50:31 +0000	[thread overview]
Message-ID: <20070227095031.156c74e5.khali@linux-fr.org> (raw)
In-Reply-To: <00c901c7568f$60a3ad40$0801a8c0@pinkeltje>

Hi Ivo,

On Thu, 22 Feb 2007 15:40:18 +0100, Ivo Manca wrote:
> The project plan can be found at 
> http://souchi.nl/prive/ProjectPlanDMIv1.1.pdf
> Here you can find all information about our goal. We hope you think the 
> extension
> will be useful and wanted. Suggestions and feedback are always welcome.

I've read the "Background Information" and "Objectives" parts, overall
it looks good, except that a proofread of the document would be very
welcome.

My comments:

* The primary objective mentions an "internal database". I'm not sure
what "internal" is supposed to mean here. Do you plan to embed a big
array of known motherboards inside the sensors-detect script itself?

* I'm very much in favor of command line parameters being added to
sensors-detect so that for example a non-interactive mode can be
supported. I wanted to do that myself for some time but could never
find the time to actually do it. Another useful mode would be a "quiet"
mode, which would hide all the details and present the probing summary.
It would still be interactive in that it should not overwrite the
configuration file without the user's consent, so it would be somewhat
intermediate between the current mode and a fully automated mode.

BTW, I think you really mean "first objective" and "second objective",
not primary and secondary.

-- 
Jean Delvare


  parent reply	other threads:[~2007-02-27  8:50 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-22 14:40 [lm-sensors] Sensors-detect with DMI detection Ivo Manca
2007-02-22 17:19 ` Jean Delvare
2007-02-23  7:08 ` Hans de Goede
2007-02-27  8:50 ` Jean Delvare [this message]
2007-02-27 10:59 ` Ivo Manca
2007-02-27 11:43 ` Jean Delvare
2007-02-27 12:24 ` Ivo Manca
2007-02-27 20:41 ` Rudolf Marek
2007-02-28 13:35 ` Hans de Goede
2007-02-28 13:44 ` Jasper Alias
2007-03-01  7:46 ` Jean Delvare
2007-03-01  8:13 ` Hans de Goede
2007-03-01  8:54 ` Jean Delvare
2007-03-16 13:06 ` Ivo Manca
2007-03-18 20:29 ` Jean Delvare
2007-03-20 12:39 ` Ivo Manca
2007-03-22 10:09 ` Ivo Manca
2007-03-23  8:59 ` Jean Delvare
2007-03-23  9:06 ` Hans de Goede
2007-03-23 15:43 ` Jean Delvare

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=20070227095031.156c74e5.khali@linux-fr.org \
    --to=khali@linux-fr.org \
    --cc=lm-sensors@vger.kernel.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.
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.