The following commit has been merged in the master branch:
commit 64b29c0010a90d1f8c3b346866db23744adc7657
Author: Adam D. Barratt <[EMAIL PROTECTED]>
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
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
