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

Bug#580249: marked as done (lintian: maintainer-script-without-set-e postinst (possible false positive))

Your message dated Tue, 04 May 2010 13:28:52 -0500
with message-id <4be06767.1608c00a.35e9.ffffc3b6@mx.google.com>
and subject line Re: Bug#580249: lintian: maintainer-script-without-set-e postinst (possible false positive)
has caused the Debian Bug report #580249,
regarding lintian: maintainer-script-without-set-e postinst (possible false positive)
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.)


-- 
580249: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=580249
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: lintian
Version: 2.4.1
Severity: minor


This message:

     P: xtrlock: maintainer-script-without-set-e postinst

Compare the file content:

     $ nl -ba debian/postinst 
     1  #!/bin/sh -e
     2  if [ -x /usr/bin/update-menus ] ; then update-menus ; fi


-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-3-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_DK.UTF-8, LC_CTYPE=en_DK.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages lintian depends on:
ii  binutils               2.20.1-8          The GNU assembler, linker and bina
ii  diffstat               1.47-1            produces graph of changes introduc
ii  dpkg-dev               1.15.7.1          Debian package development tools
ii  file                   5.04-2            Determines file type using "magic"
ii  gettext                0.17-10           GNU Internationalization utilities
ii  intltool-debian        0.35.0+20060710.1 Help i18n of RFC822 compliant conf
ii  libapt-pkg-perl        0.1.24            Perl interface to libapt-pkg
ii  libclass-accessor-perl 0.34-1            Perl module that automatically gen
ii  libipc-run-perl        0.89-1            Perl module for running processes
ii  libparse-debianchangel 1.1.1-2           parse Debian changelogs and output
ii  libtimedate-perl       1.2000-1          collection of modules to manipulat
ii  liburi-perl            1.54-1            module to manipulate and access UR
ii  locales                2.10.2-6          Embedded GNU C Library: National L
ii  man-db                 2.5.7-2           on-line manual pager
ii  perl [libdigest-sha-pe 5.10.1-12         Larry Wall's Practical Extraction 

lintian recommends no packages.

Versions of packages lintian suggests:
pn  binutils-multiarch            <none>     (no description available)
ii  libtext-template-perl         1.45-1     Text::Template perl module
ii  man-db                        2.5.7-2    on-line manual pager

-- no debconf information

--- End Message ---
--- Begin Message --- 
Jari Aalto wrote:

> Package: lintian
> Version: 2.4.1
> Severity: minor
> 
> 
> This message:
> 
>      P: xtrlock: maintainer-script-without-set-e postinst
> 
> Compare the file content:
> 
>      $ nl -ba debian/postinst
>      1  #!/bin/sh -e
>      2  if [ -x /usr/bin/update-menus ] ; then update-menus ; fi
> 

That's perfectly ok, please read the tag's description:

$ lintian-info -t maintainer-script-without-set-e
> N: maintainer-script-without-set-e
> N:
> N:   The maintainer script passes -e to the shell on the #! line rather
> N:   than using set -e in the body of the script. This is fine for normal
> N:   operation, but if the script is run by hand with sh /path/to/script
> N:   (common in debugging), -e will not be in effect. It's therefore 
better
> N:   to use set -e in the body of the script.
> N:
> N:   Refer to Debian Policy Manual section 10.4 (Scripts) for details.
> N:
> N:   Severity: pedantic, Certainty: certain
> N:
> 

Cheers,
-- 
Raphael Geissert - Debian Developer
www.debian.org - get.debian.net

--- End Message ---
Reply to: