Quoting Jean-Pierre Pourrez
>         DISTVER=`cat /etc/mandrake-release  | perl -ne 'print $1 if
>                              (s!^.*release[^\d]*(\d+(\.\d+)?).*!is)'`

Oui on gagne quelques lignes.
Par contre on s'efforce de ne pas utiliser awk, gawk et perl dans les codes
communs à toutes les distribs pour diminuer les dépendances de construction (et
le support qui va avec sur le forum).

Donc la même chose avec du sed si c'est pas plus compliqué ;-)


> Autre suggestion pour dsp-dir:
>
> # ===== arg: dsp-dir =====
> AC_ARG_WITH(dsp-dir,
>         [  --with-dsp-dir             Path to DSP files (default={datadir})],
>         DSPDIR=${withval},
>         DSPDIR=""
>         )
> # if test -z "${DSPDIR}" ; then
> #       DSPDIR=${CONFDIR}/eagle-usb/dsp
> # fi
> [ -z "${DSPDIR}" ] && DSPDIR=${datadir}/eagle-usb
>
> Sur Debian, dans la doc http://localhost/doc/autoconf/autoconf.html,
> ${datadir} semble un dossier plus approprié:
> datadir
>   The directory for installing read-only architecture-independent data.

C'est exact mais ce n'est pas une doc spécifique à Debian. Donc je vais plus
loin et je propose la suppression de @dspdir@ qui est redondant avec @datadir@
(et puis on n'installe plus jamais les codes DSP dans /etc/eagle-usb).

D'ailleurs je crois que les fichiers de lang devraient aussi tomber là...
      EU_DSP_DIR=$(DESTDIR)@datadir@      # Makefile.common
      EU_LANG_DIR=$(DESTDIR)@datadir@

mcoolive.

Reply via email to