On Wed, Apr 18, 2012 at 12:22 PM, Jean Delvare <khali@xxxxxxxxxxxx> wrote:
On Wed, 18 Apr 2012 11:20:40 +0100, Martin Suchanek wrote:My bad, I should have tested more carefully. I tested the command
> This '/libiconv\.so$' gave me error "/bin/sh: Syntax error: Unterminated
> quoted string". I do not know why ?
outside of "make" and it worked, but "make" interprets '$' so it needs
special care.
Updated patch follows, hopefully it will work OK this time:
+++ lm-sensors/prog/sensors/Module.mk 2012-04-18 13:17:51.978110528 +0200
---
prog/sensors/Module.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- lm-sensors.orig/prog/sensors/Module.mk 2008-03-26 14:37:44.000000000 +0100
@@ -37,7 +37,7 @@ INCLUDEFILES += $(PROGSENSORSSOURCES:.c=+LIBICONV := $(shell if /sbin/ldconfig -p | grep -q '/libiconv\.so$$' ; then echo \-liconv; else echo; fi)
REMOVESENSORSBIN := $(patsubst $(MODULE_DIR)/%,$(DESTDIR)$(BINDIR)/%,$(PROGSENSORSTARGETS))
REMOVESENSORSMAN := $(patsubst $(MODULE_DIR)/%,$(DESTDIR)$(PROGSENSORSMAN1DIR)/%,$(PROGSENSORSMAN1FILES))
-LIBICONV := $(shell if /sbin/ldconfig -p | grep -q libiconv\\.so ; then echo \-liconv; else echo; fi)
$(PROGSENSORSTARGETS): $(PROGSENSORSSOURCES:.c=.ro) lib/$(LIBSHBASENAME)
$(CC) $(EXLDFLAGS) -o $@ $(PROGSENSORSSOURCES:.c=.ro) $(LIBICONV) -Llib -lsensors
compiled/installed succesfully.
Should I test it by some way ?
(
There is one warning within sudo make install:
******************************************************************************
Warning: Library directory /usr/local/lib is not in /etc/ld.so.conf!
Add it and run /sbin/ldconfig for the userspace tools to work.
******************************************************************************
I have checked /etc/ld.so.conf and it contains include /etc/ld.so.conf.d/*.conf which actually include /usr/local/bin in /etc/ld.so.conf.d/libc.conf
)
cheers,
Martin
Should I test it by some way ?
(
There is one warning within sudo make install:
******************************************************************************
Warning: Library directory /usr/local/lib is not in /etc/ld.so.conf!
Add it and run /sbin/ldconfig for the userspace tools to work.
******************************************************************************
I have checked /etc/ld.so.conf and it contains include /etc/ld.so.conf.d/*.conf which actually include /usr/local/bin in /etc/ld.so.conf.d/libc.conf
)
cheers,
Martin
Jean Delvare--
_______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors