xorg/driver/xf86-input-synaptics: [PATCH 2/2] Janitor: compile warnings and "missing" symbols.
Paulo César Pereira de Andrade
pcpa at mandriva.com.br
Fri Feb 6 09:24:05 PST 2009
Daniel Stone wrote:
> On Fri, Feb 06, 2009 at 05:50:58AM -0800, Dan Nicholson wrote:
>> This is probably fine, but it's a little safer to use the
>> AC_CHECK_LIBM macro from libtool.
>> @DRIVER_NAME at _drv_la_LIBADD = $(LIBM)
> Oooh, cool. We've got a few hardcoded -lm's around.
In this specific case, it should not really matter much
as the X Server must be linked with libm.
But for example, I believe xf86-input-acecad is better
@DRIVER_NAME at _drv_la_LIBADD = -lsysfs
then the hack it had previously to dlopen libsysfs.
The @DRIVER_NAME@ patterns probably would be better
also not used. As it is very unlikely a single Makefile.am
could be used for all drivers, and I don't see it much
useful, other then if someone wants to give a different
name to a driver every week. But then some work must be
done to expand @DRIVER_NAME@ in the C file also...
More information about the xorg