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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 02/22/2010 11:40 PM, VDR User wrote:
On Mon, Feb 22, 2010 at 1:26 PM, Hans de Goede<hdegoede@xxxxxxxxxx>  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:
<some-path>/board_vendor/board_name

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.

Regards,

Hans

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux