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

Re: dh_config_model_upgrade: package upgrade with Config::Model



On Thu, Dec 17, 2009 at 8:24 AM, Dominique Dumont
<dominique.dumont@hp.com> wrote:
> On Wednesday 16 December 2009 17:40:55 Neil Williams wrote:
>> No. The package should simply exit cleanly with a successful return
>> value if perl does not exist, letting everything else proceed as before.
>> The postinst itself needs to check - that way, Emdebian doesn't have to
>> patch every package using dh_config_model.
>
> Ok. Here's the new postinst snippet injected by dh_config_model_upgrade:
>
> # In case of error (error in configuration file or model bug), the
> # configuration file is left as is.
>
> # testing perl is required to avoid problem in embedded environments
> if [[ -e /usr/bin/perl ]]

'[[' for testing is a bashism.  This should be

  if [ -e /usr/bin/perl ]

or more accurately

  if [ -x /usr/bin/perl ]

-- 
James
GPG Key: 1024D/61326D40 2003-09-02 James Vega <jamessan@debian.org>


Reply to: