[PATCH] detect path to dmidecode in configure (fd.o bug #9318)

Danny Kukawka danny.kukawka at web.de
Fri Feb 16 11:51:43 PST 2007


On Thursday 15 February 2007 20:21, David Zeuthen wrote:
> On Fri, 2007-02-09 at 19:43 +0100, Danny Kukawka wrote:
> > this patch adds detection of the location of dmidecode to configure. If
> > dmidecode could not get found we fall back to the same path as already
> > hardcoded in probe-smbios.c. This would fix fd.o bug #9318
>
> I guess that's fine although I'm not too fond of requiring the tool to
> be available during build (e.g. it forces a BuildRequires in the RPM
> spec file). But I suppose it's useful for some people. Please commit.
> Thanks.

The alternative is the attached patch which check the path to dmidecode on 
runtime. This should be no really problem with performance since the addon 
get started only one time - on hald startup.

I can live also with this patch.

Danny

 probe-smbios.c |   27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-detect-dmidecode_on_runtime.diff
Type: text/x-diff
Size: 1616 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20070216/32266115/hal-detect-dmidecode_on_runtime.bin


More information about the hal mailing list