[SCM] Debian package checker branch, master, updated. 1.24.3-5-g64b29c0
The following commit has been merged in the master branch:
commit 64b29c0010a90d1f8c3b346866db23744adc7657
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date: Sat Aug 9 19:32:40 2008 +0100
Improve the tested cases for read bashisms
diff --git a/testset/maintainer-scripts/debian/prerm b/testset/maintainer-scripts/debian/prerm
index 55f21b5..2b6cd99 100644
--- a/testset/maintainer-scripts/debian/prerm
+++ b/testset/maintainer-scripts/debian/prerm
@@ -172,3 +172,10 @@ foobar@()
{
# As is this
}
+
+# This is ok
+read -r foo
+# but these aren't
+read -r
+read -p "Would you like to restart the service?" foo bar
+read --fish
diff --git a/testset/tags.maintainer-scripts b/testset/tags.maintainer-scripts
index bcab8c3..0cfc9c9 100644
--- a/testset/tags.maintainer-scripts
+++ b/testset/tags.maintainer-scripts
@@ -108,6 +108,9 @@ W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:155 'sh +O'
W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:158 ' {a,b}'
W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:166 ' foo.bar()'
W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:171 'foobar@() '
+W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:179 'read -r'
+W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:180 'read -p'
+W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:181 'read --fish'
W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:19 '[ "$2" == '
W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:23 'function foo('
W: maintainer-scripts: possible-bashism-in-maintainer-script prerm:26 '&>'
@@ -145,6 +148,10 @@ W: maintainer-scripts: postinst-should-not-set-usr-doc-link
W: maintainer-scripts: postrm-does-not-purge-debconf
W: maintainer-scripts: postrm-has-useless-call-to-ldconfig
W: maintainer-scripts: read-in-maintainer-script postinst:20
+W: maintainer-scripts: read-in-maintainer-script prerm:177
+W: maintainer-scripts: read-in-maintainer-script prerm:179
+W: maintainer-scripts: read-in-maintainer-script prerm:180
+W: maintainer-scripts: read-in-maintainer-script prerm:181
W: maintainer-scripts: read-in-maintainer-script prerm:62
W: maintainer-scripts: read-in-maintainer-script prerm:63
W: maintainer-scripts: read-in-maintainer-script prerm:64
--
Debian package checker
Reply to: