[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#681280: kernel warning at .../net/sched/sch_generic.c:255 dev_watchdog - eth0 (sky2) transmit queue 0 timed out



On Tue, Oct 30, 2012 at 10:46:29AM +0100, Uwe Kleine-König wrote:
> > -eval "set -- $(getopt -n "$0" -- "f:j:s:" "$@")"
> > +if [ -z "${DEBIAN_KERNEL_DISABLE_DEBUG+set}" ]; then
> I don't know that construct, only :+, but + seems to do the same without
> me finding it in the manpage of neither bash nor dash.

In the bash(1) manpage, in the "Parameter Expansion" section, it says:

  When not performing substring expansion, using the forms documented
  below, bash tests for a parameter that is unset or null.  Omitting
  the colon results in a test only for a parameter that is unset.

Oh, OK, so my earlier suggestion of replacing this with just
if [ -z "$DEBIAN_KERNEL_DISABLE_DEBUG" ]; then
is incorrect: the above test will leave the value of
DEBIAN_KERNEL_DISABLE_DEBUG alone if it is set (even if it is set to
null), but will set it to "yes, ..." if it is unset.

   Julian


Reply to: