Quelques modifs auxquels j'ai pensé. 1/ Remplacer partout les if($DISTRIB ==... par des constructions de la forme case $DISTRIB... C'est plus facile à manier quand on veut ajouter une nouvelle distribution
2/ L'ajout de $SYSCONF_DIR est bien compliqué. Je propose de supprimer $SYSCONF_DIR et de remplacer mkdir -p $SYSCONF_DIR par ceci mkdir -p `dirname $SYSCONF_FILE` Ou bien de ne plus calculé $SYSCONF_FILE dans le ./configure, et dans setvars on a SYSCONF_FILE=$SYSCONF_DIR/eagle-usb. 3/ Améliorer configure.in 3.1/ Il semblerait que les macros AC_CHECK_PROG soit plutôt pensé pour les programmes nécessaires à la compilation locale. Dans le cas de la construction de paquet, beaucoup de ces vérifications ne sont plus pertinentes. Je propose de bien séparer ce qui est nécessaire à la compilation de ce qui sera nécessaire à l'exécution, pour les seconds on pourrait ajouter une option pour ignorer leur absence. 3.2/ AC_OUTPUT(Makefile.common) est obsolète (autoconf -Wall le signale). Il faut utiliser AC_CONFIG_FILES([Makefile.common]) AC_OUTPUT() 3.3/ AC_INIT(driver/eu_main.c) Sous réserve que j'ai bien compris la doc (http://www.gnu.org/software/autoconf/manual/autoconf-2.57/html_node/autoconf.html) il faudrait plutôt écrire quelque chose comme AC_PREREQ(2.52) AC_INIT(eagle-usb, 2.0.0) AC_CONFIG_SRCDIR(driver/eu_main.c) 3.4/ L'utilisation de AC_CHECK_FILE provoque les Warnings à propos de la compilation croisée. Je devine (la doc est pas très clair) que c'est parce que c'est prévu pour vérifier la présence d'un fichier qui n'appartient pas aux sources. Le plus simple ici est de ne pas utiliser cette macro et de faire un simple test shell pour vérifier la présence de la doc. 3.5/ Prévoir un `sous-configure' pour eagleconnec. Pour l'instant les vérifications concernant eagleconnec sont commentées, mais il serait idiot de bloquer sur un manque pour la génération d'eagleconnec alors que l'on ne veut pas l'utiliser #AC_CONFIG_SUBDIRS(utils/eagleconnec) #nouvelles options --with[out]-eagleconnec J'ai d'autres modifications relevant plus du cosmétique. Si tu es d'accord sur les grandes lignes, je t'envoie un diff du configure.in. mcoolive.
