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 | 71e5b11e49283bd59a96bf04f88fe3ea45b9396f (patch) | |
| tree | bee7f3c72bf20e407335eb690d4a551426edc241 /pkgtools/pkglint | |
| parent | 99efcfdd7a7f3dee753727a5b309abb8ee4526a3 (diff) | |
| download | pkgsrc-71e5b11e49283bd59a96bf04f88fe3ea45b9396f.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/pkglint')
| -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", |