All of lore.kernel.org
 help / color / mirror / Atom feed
* lm_sensors2/kernel/include sensors.h
@ 2005-05-19  6:24 Mark D. Studebaker 
  2005-05-19  6:24 ` Mark D. Studebaker 
  2005-05-19  6:24 ` Jean Delvare
  0 siblings, 2 replies; 3+ messages in thread
From: Mark D. Studebaker  @ 2005-05-19  6:24 UTC (permalink / raw)
  To: lm-sensors

Don't do this!
sensors.h is now automatically generated...
(see sensors.h.template and Module.mk).
We're going to remove sensors.h from CVS before the release.
If there's a warning to be fixed it's in sensors.h.template??

author@mordac.netroedge.com wrote:
> Update of /home/cvs/lm_sensors2/kernel/include
> In directory mordac.netroedge.com:/tmp/cvs-serv26137
> 
> Modified Files:
> 	sensors.h 
> Log Message:
> (Khali) Trivial warning removal.
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* lm_sensors2/kernel/include sensors.h
  2005-05-19  6:24 lm_sensors2/kernel/include sensors.h Mark D. Studebaker 
  2005-05-19  6:24 ` Mark D. Studebaker 
@ 2005-05-19  6:24 ` Jean Delvare
  1 sibling, 0 replies; 3+ messages in thread
From: Jean Delvare @ 2005-05-19  6:24 UTC (permalink / raw)
  To: lm-sensors


> Don't do this!
> sensors.h is now automatically generated...
> (see sensors.h.template and Module.mk).
> We're going to remove sensors.h from CVS before the release.
> If there's a warning to be fixed it's in sensors.h.template??

Isn't removing sensors.h right before the release a bit dangerous? The
autogeneration feature should be tested before that.

As for the warning I removed, it's in a comment that does not exist in
sensors.h.template, so there's nothing to care about.

-- 
Jean Delvare
http://www.ensicaen.ismra.fr/~delvare/

^ permalink raw reply	[flat|nested] 3+ messages in thread

* lm_sensors2/kernel/include sensors.h
  2005-05-19  6:24 lm_sensors2/kernel/include sensors.h Mark D. Studebaker 
@ 2005-05-19  6:24 ` Mark D. Studebaker 
  2005-05-19  6:24 ` Jean Delvare
  1 sibling, 0 replies; 3+ messages in thread
From: Mark D. Studebaker  @ 2005-05-19  6:24 UTC (permalink / raw)
  To: lm-sensors

It may be dangeroous (but I don't think so) but it's already happened. 
sensors.h is overwritten by the automatically-generated version.
Looks to me like it works great.
Keep testing...

Jean Delvare wrote:
>>Don't do this!
>>sensors.h is now automatically generated...
>>(see sensors.h.template and Module.mk).
>>We're going to remove sensors.h from CVS before the release.
>>If there's a warning to be fixed it's in sensors.h.template??
> 
> 
> Isn't removing sensors.h right before the release a bit dangerous? The
> autogeneration feature should be tested before that.
> 
> As for the warning I removed, it's in a comment that does not exist in
> sensors.h.template, so there's nothing to care about.
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-05-19  6:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-05-19  6:24 lm_sensors2/kernel/include sensors.h Mark D. Studebaker 
2005-05-19  6:24 ` Mark D. Studebaker 
2005-05-19  6:24 ` Jean Delvare

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.