Selon JoDaY <[EMAIL PROTECTED]>:

> Bonjour,
>
> Serait il possible de rajouter pump:
> fichier configure l:5490 eagle-usb-2.0.0_20041012snapshot
>
> # ===== check for dhclient =====
> for ac_prog in dhclient dhcpcd pump
>

It's not only for debian packages, so I reply on the ml.

Yes we can add the “pump” choice during ./configure process.
But there is a bug...

We forget to use DHCP variable in other script...
   # extract of eagleusb/configure
   AC_SUBST(DHCP)
And we use always “dhclient”.
   # extract of eagleusb/utils/scripts/fctStartAdsl
   dhclient $INTERFACE &>/dev/null

In other hand, it's a bad idea to choose pump or dhclient during the
compilation. If I had both and I remove one...
So I think the best solution is to modify eagleusb/configure as you describe
(we can keep a static check) and do not export DHCP variable.
But setvars may choose a chcp client which is present.

  # proposal for eagleusb/utils/scripts/setvars
  for prog in dhclient3 dhclient dhcpcd pump; do
     if which $prog >/dev/null; then
        DHCP=$prog
        break
     fi
  done
  if [ x$DHCP = x ]; then echo "Error: No DHCP Client"; exit 1; fi

mcoolive.

Reply via email to