On Wed, Nov 26, 2008 at 10:01 AM, Martin Michlmayr <[EMAIL PROTECTED]> wrote:
> Package: micro-evtd-udeb
> Version: 3.3.3-6+lenny2
> Severity: minor
> Tags: patch
>
> I get the following:
>
> ~ # micro_evtd.command init
> sleep: invalid number '0.030'
>
> The problem is that busybux sleep only accepts integers, but microapl
> converts the argument to a float.  Using 1000 also won't work:
>
> ~ # /usr/sbin/microapl -a bz_melody 30 b4
> sleep: invalid number '0.030'
> ~ # /usr/sbin/microapl -a bz_melody 1000 b4
> sleep: invalid number '1.000'
>
> I don't think it really matters much since beeping works fine... and
> micro_evtd.command doesn't contain "set -e", so the error is ignored
> anyway.  However, just to be on the safe side, I suggest you add a ||
> true after the microapl calls that involve bz_melody.  There's only
> one anyway:
>
> --- debian/udeb/micro_evtd.command~     2008-11-26 10:00:45.000000000 +0100
> +++ debian/udeb/micro_evtd.command      2008-11-26 10:01:12.000000000 +0100
> @@ -39,7 +39,7 @@
>                ;;
>        init)
>                $MICROAPL led_set_blink 0
> -               $MICROAPL bz_melody 30 b4
> +               $MICROAPL bz_melody 30 b4 || true
>                ;;
>        start)
>                # Start micro_evtd if not already running, exit with failure

I stumbled upon this as well (but obviously just
ignored because the script worked). You have a
valid remark. Thanks for your patch!

Do you think it is necessary to fix this for Lenny?


-- Per



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to