The attached patch fixes WvModem::setup_modem to not treat it as a fatal
error when non-root users are unable to use the TIOCSSERIAL ioctl.
Incidentally, the other use of HAVE_LINUX_SERIAL_H in streams/wvmodem.cc
looks quite broken: there is no reason that declaration of the cfmakeraw()
function should be correlated with the presence of the linux/serial.h
header. Please add a proper autoconf check to detect cfmakeraw() support
separately instead of tacking it onto the check for a completely unrelated
header.
tags 292618 +patch
thanks
The attached patch fixes WvModem: :setup_ modem to not treat it as a fatal
error when non-root users are unable to use the TIOCSSERIAL ioctl.
Incidentally, the other use of HAVE_LINUX_SERIAL_H in streams/wvmodem.cc
looks quite broken: there is no reason that declaration of the cfmakeraw()
function should be correlated with the presence of the linux/serial.h
header. Please add a proper autoconf check to detect cfmakeraw() support
separately instead of tacking it onto the check for a completely unrelated
header.
--
Steve Langasek
postmodern programmer