diff options
| author | rillig <rillig@pkgsrc.org> | 2006-07-18 21:13:22 +0000 |
|---|---|---|
| committer | rillig <rillig@pkgsrc.org> | 2006-07-18 21:13:22 +0000 |
| commit | 52fd2bb1b7372b214d8fceda24cfe5e8ca15abad (patch) | |
| tree | bee7f3c72bf20e407335eb690d4a551426edc241 /pkgtools | |
| parent | d50e117e5b978883d2610447de6fab499213c198 (diff) | |
| download | pkgsrc-52fd2bb1b7372b214d8fceda24cfe5e8ca15abad.tar.gz | |
If, in a "case" statement of a shell command, foo|bar appears, this is
not a pipe, so don't warn about an ignored exitcode.
Diffstat (limited to 'pkgtools')
| -rw-r--r-- | pkgtools/pkglint/files/pkglint.pl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pkgtools/pkglint/files/pkglint.pl b/pkgtools/pkglint/files/pkglint.pl index 5de8ec2c907..2e07e10e577 100644 --- a/pkgtools/pkglint/files/pkglint.pl +++ b/pkgtools/pkglint/files/pkglint.pl @@ -1,5 +1,5 @@ #! @PERL@ -# $NetBSD: pkglint.pl,v 1.656 2006/07/18 20:37:31 rillig Exp $ +# $NetBSD: pkglint.pl,v 1.657 2006/07/18 21:13:22 rillig Exp $ # # pkglint - static analyzer and checker for pkgsrc packages @@ -4273,7 +4273,7 @@ sub checkline_mk_shelltext($$) { $line->log_warning("Please use \${ECHO_N} instead of \"echo -n\"."); } - if ($opt_warn_extra && $shellword eq "|") { + if ($opt_warn_extra && $state != SCST_CASE_LABEL_CONT && $shellword eq "|") { $line->log_warning("The exitcode of the left-hand-side command of the pipe operator is ignored."); $line->explain_warning( "If you need to detect the failure of the left-hand-side command, use", |