This is an automated email from the git hooks/post-receive script. guillem pushed a commit to branch master in repository dpkg.
View the commit online: https://git.dpkg.org/cgit/dpkg/dpkg.git/commit/?id=f50aea672bab786d75d51dcbac47fc1ee9703bc8 The following commit(s) were added to refs/heads/master by this push: new f50aea672 test: Print WARN instead of FAIL for maintscript fd leaks f50aea672 is described below commit f50aea672bab786d75d51dcbac47fc1ee9703bc8 (HEAD -> master) Author: Guillem Jover <guil...@debian.org> AuthorDate: Fri Jan 8 19:36:40 2021 +0100 test: Print WARN instead of FAIL for maintscript fd leaks Whether we have file descriptor leaks is also dependent on processes calling dpkg, so we should not print these as failures, as that is confusing and we might not be able to solve these. --- tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postinst | 2 +- tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postrm | 2 +- tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/preinst | 2 +- tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/prerm | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postinst b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postinst index 4c31ba4b6..580884db1 100755 --- a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postinst +++ b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postinst @@ -12,5 +12,5 @@ done if [ -n $fddir ]; then ls -l $fddir # We should have the 3 standard descriptors, plus 1 for the open directory. - [ `ls $fddir | wc -l` -eq 4 ] || echo FAIL: maintscript leaks fd + [ `ls $fddir | wc -l` -eq 4 ] || echo WARN: maintscript leaks fd fi diff --git a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postrm b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postrm index 4c31ba4b6..580884db1 100755 --- a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postrm +++ b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/postrm @@ -12,5 +12,5 @@ done if [ -n $fddir ]; then ls -l $fddir # We should have the 3 standard descriptors, plus 1 for the open directory. - [ `ls $fddir | wc -l` -eq 4 ] || echo FAIL: maintscript leaks fd + [ `ls $fddir | wc -l` -eq 4 ] || echo WARN: maintscript leaks fd fi diff --git a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/preinst b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/preinst index 4c31ba4b6..580884db1 100755 --- a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/preinst +++ b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/preinst @@ -12,5 +12,5 @@ done if [ -n $fddir ]; then ls -l $fddir # We should have the 3 standard descriptors, plus 1 for the open directory. - [ `ls $fddir | wc -l` -eq 4 ] || echo FAIL: maintscript leaks fd + [ `ls $fddir | wc -l` -eq 4 ] || echo WARN: maintscript leaks fd fi diff --git a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/prerm b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/prerm index 4c31ba4b6..580884db1 100755 --- a/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/prerm +++ b/tests/t-maintscript-leak/pkg-maintscript-leak/DEBIAN/prerm @@ -12,5 +12,5 @@ done if [ -n $fddir ]; then ls -l $fddir # We should have the 3 standard descriptors, plus 1 for the open directory. - [ `ls $fddir | wc -l` -eq 4 ] || echo FAIL: maintscript leaks fd + [ `ls $fddir | wc -l` -eq 4 ] || echo WARN: maintscript leaks fd fi -- Dpkg.Org's dpkg
