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

Bug#772316: marked as done (kdevplatform8-libs: bashism in /bin/sh script)

Your message dated Sun, 07 Aug 2016 17:51:45 +0200
with message-id <2102380.SQhROSZtVZ@bagend>
and subject line Fixed upstream by specifying /bin/bash instead
has caused the Debian Bug report #772316,
regarding kdevplatform8-libs: bashism in /bin/sh script
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.)


-- 
772316: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=772316
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: kdevplatform8-libs
Severity: important
Version: 1.7.0-2
User: debian-release@lists.debian.org
Usertags: goal-dash

Hi,

I've ran checkbashisms (from the 'devscripts' package) over the whole
archive and I found that your package has a /bin/sh script that uses a
"bashism".

checkbashisms' output:
> possible bashism in ./usr/bin/kdev_format_source line 61 (should be 'b =
> a'):
> while ! [ "$(pwd)" == "/" ]; do


Not using bash (or a Debian Policy compliant shell interpreter that doesn't
provide such an extra feature) as /bin/sh is likely to lead to errors or
unexpected behaviours. Please be aware that dash is the default /bin/sh.

Please closely examine the above output and the script, and determine
what the proper severity of the bug is, and adjust it accordingly. If
it's important or greater please hurry to get this fixed for jessie.

Hints about how to fix bashisms can be found at:
https://wiki.ubuntu.com/DashAsBinSh

Thanks in advance,
Raphael Geissert

--- End Message ---
--- Begin Message --- 
Version: 1.7.1-1

This has been fixed upstream [1] and that is incorporated into Debian [2] (but 
it was earlier fixing in the Debian package [3]), thus closing this bug.

[1] https://quickgit.kde.org/?
p=kdevplatform.git&a=commit&h=04a3b04e60275aa58f2c3b0ef8c07ddc613a5b6c
[2] https://anonscm.debian.org/cgit/pkg-kde/kde-std/kdevplatform.git/commit/?
id=cd6306d9ad212cd6f4a0a8200f7a056b0f2ec617
[3] https://anonscm.debian.org/cgit/pkg-kde/kde-std/kdevplatform.git/commit/?
id=6dcec164b888b146ee6a0e9edf952d9cb12fb0bf

Attachment: signature.asc
Description: This is a digitally signed message part.


--- End Message ---
Reply to: