diff options
| author | rillig <rillig> | 2006-07-18 21:13:22 +0000 |
|---|---|---|
| committer | rillig <rillig> | 2006-07-18 21:13:22 +0000 |
| commit | f4755a28e4ce47d6dea38ffc48226f35646091a4 (patch) | |
| tree | bee7f3c72bf20e407335eb690d4a551426edc241 /pkgtools/pkglint | |
| parent | ffbe9a06fa257b6075288f245e70932a31b15b80 (diff) | |
| download | pkgsrc-f4755a28e4ce47d6dea38ffc48226f35646091a4.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", |