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

Bug#866287: marked as done (debbugs: maintainer script(s) not using strict mode)



Your message dated Wed, 28 Jun 2017 14:47:07 -0500
with message-id <20170628194706.vivcqpbw63x2avnx@geta>
and subject line Re: Bug#866287: debbugs: maintainer script(s) not using strict mode
has caused the Debian Bug report #866287,
regarding debbugs: maintainer script(s) not using strict mode
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 this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
866287: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=866287
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: debbugs
Version: 2.4.1.1
Severity: normal
User: treinen@debian.org
Usertags: colis-shparser

Dear maintainer,

at least one of the maintainer scripts (preinst, postinst, prerm,
postrm) of the package debbugs does not use strict mode. Policy
section 10.4 says:

"Shell scripts (sh and bash) [..] should almost certainly start with
set -e so that errors are detected."

"Every script should use set -e or check the exit status of every
command."

Please insert a "set -e" at the beginning of your script to enable
strict mode. You should not replace this by a first line "#!/bin/sh
-e" as it is not effective when your script is executed by an
explicit invocation of sh.

Note that this might make your script fail in cases where it did not
fail before. This is the purpose of strict mode - make it fail when
any unexpected error is encountered. You should make sure that you
catch any error (non-zero exit codes of commands) that you decide to
tolerate. Techniques to locally catch an error include using
appropriate options to your command when available, adding a " ||
true" at the end of the command, or selectively switching off strict
mode by "set +e" and switching it back on again later by "set -e". 

This bug filing has been discussed and approved in thread [1].

-Ralf.

[1] https://lists.debian.org/debian-devel/2017/06/msg00342.html 

--- End Message ---
--- Begin Message ---
Source: debbugs
Version: 2.4.2~exp1

On Wed, 28 Jun 2017, treinen@debian.org wrote:
> at least one of the maintainer scripts (preinst, postinst, prerm,
> postrm) of the package debbugs does not use strict mode. Policy
> section 10.4 says:

Debbugs version 2.4.2~exp1 no longer ships any maintainer scripts.


-- 
Don Armstrong                      https://www.donarmstrong.com

Whatever you do will be insignificant, but it is very important that
you do it.
 -- Mohandas Karamchand Gandhi

--- End Message ---

Reply to: