diff options
| author | wiz <wiz> | 2012-07-17 19:24:46 +0000 |
|---|---|---|
| committer | wiz <wiz> | 2012-07-17 19:24:46 +0000 |
| commit | 9035cd0ddd35e993ee8fabf490a9999a383da3dc (patch) | |
| tree | afaa813a3dde26a0e31fda4b40fbdcaa6da22429 /pkgtools/pkglint/files | |
| parent | bdc17b8250a8a473a87a7e738a8771f92f73d7bf (diff) | |
| download | pkgsrc-9035cd0ddd35e993ee8fabf490a9999a383da3dc.tar.gz | |
Make missing comments in patches an error. Bump version.
The guide prescribes it for quite some time and it's common courtesy
for packages modifying packages after you.
ok gdt@
Diffstat (limited to 'pkgtools/pkglint/files')
| -rw-r--r-- | pkgtools/pkglint/files/pkglint.pl | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/pkgtools/pkglint/files/pkglint.pl b/pkgtools/pkglint/files/pkglint.pl index 9777d6378b9..986536b55fb 100644 --- a/pkgtools/pkglint/files/pkglint.pl +++ b/pkgtools/pkglint/files/pkglint.pl @@ -1,5 +1,5 @@ #! @PERL@ -# $NetBSD: pkglint.pl,v 1.841 2012/07/14 20:32:37 wiz Exp $ +# $NetBSD: pkglint.pl,v 1.842 2012/07/17 19:24:46 wiz Exp $ # # pkglint - static analyzer and checker for pkgsrc packages @@ -7415,12 +7415,12 @@ sub checkfile_patch($) { # }], [PST_TEXT, re_patch_cfd, PST_CFA, sub() { if (!$seen_comment) { - $line->log_warning("Comment expected."); + $line->log_error("Comment expected."); } $line->log_warning("Please use unified diffs (diff -u) for patches."); }], [PST_TEXT, re_patch_ufd, PST_UFA, sub() { if (!$seen_comment) { - $line->log_warning("Comment expected."); + $line->log_error("Comment expected."); } }], [PST_TEXT, re_patch_text, PST_TEXT, sub() { $seen_comment = true; @@ -7432,14 +7432,14 @@ sub checkfile_patch($) { if ($seen_comment) { $opt_warn_space and $line->log_note("Empty line expected."); } else { - $line->log_warning("Comment expected."); + $line->log_error("Comment expected."); } $line->log_warning("Please use unified diffs (diff -u) for patches."); }], [PST_CENTER, re_patch_ufd, PST_UFA, sub() { if ($seen_comment) { $opt_warn_space and $line->log_note("Empty line expected."); } else { - $line->log_warning("Comment expected."); + $line->log_error("Comment expected."); } }], [PST_CENTER, undef, PST_TEXT, sub() { $opt_warn_space and $line->log_note("Empty line expected."); |