On Thu, 01 Jul 2010 22:11:55 +0100, Julien Cristau wrote:

> > > > +               if $(/etc/init.d/mysql status >/dev/null 2>&1); then
> > > What is this trying to test? [..]
> > Seems to work fine: [..]
> It would be clearer without the ``$( )'', IMO...

Agreed, it's not very well readable that way.

For the NMU I (and probably Vagrant too when preparing the patch)
wanted to keep the maintainer's style but maybe the maintainer can
use the proposal for clarification in a future upload.

Cheers,
gregor

-- 
 .''`.   http://info.comodo.priv.at/ -- GPG key IDs: 0x8649AA06, 0x00F3CFE4
 : :' :  Debian GNU/Linux user, admin, & developer - http://www.debian.org/
 `. `'   Member of VIBE!AT & SPI, fellow of Free Software Foundation Europe
   `-    NP: Nick Cave And The Bad Seeds: Get Ready For Love

Attachment: signature.asc
Description: Digital signature

Reply via email to