Your message dated Fri, 8 Feb 2008 14:02:09 +0100
with message-id <[EMAIL PROTECTED]>
and subject line Bug#464637: cron-apt: bashism in /bin/sh script
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--- Begin Message ---
Package: cron-apt
Severity: important
Version: 0.6.4
User: [EMAIL PROTECTED]
Usertags: goal-dash

Hello maintainer,

While performing an archive wide checkbashisms (from the 'devscripts' package) 
check I've found your package containing A/SEVERAL /bin/sh script/S making 
use of a bashism.

checkbashisms' output:
> possible bashism in ./usr/sbin/cron-apt line 294 ($RANDOM):
>         TIME=$(($RANDOM % $RUNSLEEP))


Not using bash as /bin/sh would lead to errors. Please be aware that although 
bash is currently the default /bin/sh there's a release goal for Lenny to 
make dash the default /bin/sh[1].

If you want more information about dash as /bin/sh, you can read:
http://lists.debian.org/debian-release/2008/01/msg00189.html
https://wiki.ubuntu.com/DashAsBinSh

[1]http://release.debian.org/lenny/goals.txt

Thank you,
-- 
Atomo64 - Raphael

Please avoid sending me Word, PowerPoint or Excel attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html



--- End Message ---
--- Begin Message ---
Hi

This is actually handled bu setting this veriable from /dev/urandom if
it contains an empty value.

        if [ -z "$RANDOM" ] ; then
            # A fix for shells that do not have this bash feature.
            RANDOM=$(dd if=/dev/urandom count=1 2> /dev/null | cksum | cut 
-c"1\-5")
        fi
        TIME=$(($RANDOM % $RUNSLEEP))


Thanks anyway for the report.

Best regards,

// Ola

On Thu, Feb 07, 2008 at 08:53:48PM -0600, Raphael Geissert wrote:
> Package: cron-apt
> Severity: important
> Version: 0.6.4
> User: [EMAIL PROTECTED]
> Usertags: goal-dash
> 
> Hello maintainer,
> 
> While performing an archive wide checkbashisms (from the 'devscripts' 
> package) 
> check I've found your package containing A/SEVERAL /bin/sh script/S making 
> use of a bashism.
> 
> checkbashisms' output:
> > possible bashism in ./usr/sbin/cron-apt line 294 ($RANDOM):
> >         TIME=$(($RANDOM % $RUNSLEEP))
> 
> 
> Not using bash as /bin/sh would lead to errors. Please be aware that although 
> bash is currently the default /bin/sh there's a release goal for Lenny to 
> make dash the default /bin/sh[1].
> 
> If you want more information about dash as /bin/sh, you can read:
> http://lists.debian.org/debian-release/2008/01/msg00189.html
> https://wiki.ubuntu.com/DashAsBinSh
> 
> [1]http://release.debian.org/lenny/goals.txt
> 
> Thank you,
> -- 
> Atomo64 - Raphael
> 
> Please avoid sending me Word, PowerPoint or Excel attachments.
> See http://www.gnu.org/philosophy/no-word-attachments.html
> 
> 
> 

-- 
 --------------------- Ola Lundqvist ---------------------------
/  [EMAIL PROTECTED]                     Annebergsslingan 37      \
|  [EMAIL PROTECTED]                     654 65 KARLSTAD          |
|  http://opalsys.net/                 +46 (0)70-332 1551       |
\  gpg/f.p.: 7090 A92B 18FE 7994 0C36  4FE4 18A1 B1CF 0FE5 3DD9 /
 ---------------------------------------------------------------


--- End Message ---

Reply via email to