[lm-sensors] [RFC PATCH v1 0/2] Add the sensors-config tool

Hans de Goede hdegoede at redhat.com
Tue Feb 23 09:20:32 CET 2010


On 02/22/2010 11:40 PM, VDR User wrote:
> On Mon, Feb 22, 2010 at 1:26 PM, Hans de Goede<hdegoede at redhat.com>  wrote:
>>> At the moment the tool is looking under CONFIG_DIR for a directory
>>> BOARD_VENDOR and into this directory for a file BOARD_NAME. This could
>>> be done as well as from shell script. Yes, we can use sed for regex
>>> matching but with the python way installing the automobo.conf is a
>>> one-liner encapsulated inside the config tool.
>> I understand doing the installation of automobo.conf is easier from
>> python, but we will get lots of complaints if we try to get distro
>> to carry an initscript which depends on python as a default enabled
>> initscript.
> You are right, you can expect a LOT of complaints if you make an
> initscript python-dependent.  Especially when there is no good reason
> to do so.  Being able to do it in Python with a one-liner is not even
> close to a good reason!
>> 1) How will you deal with board / produty version (we are going to need
>>    these to uniquely identify some boards)
> Not sure why this would be an issue..?

This would be an issue when using:

To locate the correct config file, as there is no version present in that scheme

>> 2) Are you going to handle having multiple board_name's in the same
>>    config file.
> Why would this be any problem at all?

The same, this would mean having multiple copies of the same file when using
the above dir layout.



More information about the lm-sensors mailing list