Commit graph

1827 commits

Author SHA1 Message Date
rillig
234e62b9d0 Updated pkglint to 4.22.
A distfile that only has an SHA1 checksum, but not an RMD160 one, is
considered an error. While at it, made the distinfo file check a little
stricter.
2005-09-27 21:13:20 +00:00
rillig
d8c6d398a6 Updated pkglint to 4.21.5.
Insist that PKGREVISION should be an integer or be undefined at all.
2005-09-27 18:58:56 +00:00
abs
c413838a65 Update pkg_chk to 1.62:
+ Implement defining tag groups in pkgchk.conf. For example
      "mailserver = tll gta tycoon jeeves iris"
    + If the first tag on a line starts with a '-', there is now an
      implicit '*' before it
    + -B no longer implies -i
2005-09-27 17:13:03 +00:00
rillig
718b487651 Updated pkglint to 4.21.4.
The last change introduced some false positive diagnostics. If a
distinfo file contains the checksum for a patch that does not exist,
this is not considered to be an error, but only a warning, as the
additional checksums do not influence the package.
2005-09-23 13:08:23 +00:00
rillig
24412e2a4e Updated pkglint to 4.21.3.
Fixed false positive errors for PHP packages.
2005-09-22 15:54:23 +00:00
rillig
5ed3de4dcc Updated pkglint to 4.21.2.
Changed the file name pattern for patch files to avoid false positives
for DISTFILES that start with "patch-" (requested by salo). Added a
warning for invalid patch file names.
2005-09-22 11:56:37 +00:00
rillig
5ea208ffc4 The diagnostics for the COMMENT variable checks include the location and
are therefore enabled by default. Requested by wiz.

Set the version to 4.21.1.
2005-09-22 01:46:46 +00:00
martti
2c976e8ade Removed distfetch from pkgsrc 2005-09-16 07:41:50 +00:00
abs
345ac26c9b update pkgtools/pkg_chk to 1.61.
Accept relative paths for -C and -P - fixes PR/31277 from Tero Kivinen
2005-09-11 10:39:03 +00:00
wiz
9c8b05d7af regen 2005-09-10 22:33:14 +00:00
wiz
2624c4e38a Date string needs no leading zeroes. 2005-09-10 22:32:54 +00:00
wiz
81184e6850 Use \*[Am] instead of & for HTML output. 2005-09-10 22:32:43 +00:00
rillig
eadbcbe2f0 This is pkglint-4.21.
Changes since 4.20 include:
 - In the diagnostics, FATAL is replaced with ERROR.
 - The command line option -g changes the format of the diagnostics to be
   similar to the one of gcc.
 - Removed false warnings when checking if PLISTS are ordered.
 - Removed the check for contiguous blank lines.
 - Added a check to PLISTs that man pages are not installed in share/man.
 - pkglint -r allows to recursively check directories.
 - The pkgsrc root directory can be checked.
 - Removed false warnings for patches that contain "---" lines.
 - Switched to checking logical lines instead of physical lines.
   Diagnostics for logical lines contain the range of physical lines
   instead of a single line number.
 - Some ../.. path components are stripped from the diagnostics.
 - When checking for the direct use of tool names, only the context of the
   use, not the whole shell command is output.
 - Patches that contain RCS Ids should not have the -ko CVS mode anymore.
 - Variables that are modified using the "+=" operator are considered to be
   lists, so they should have a name indicating a plural.
2005-09-09 12:16:29 +00:00
rillig
fe420411f3 Changed the regular expression for detecting valid .include lines in
Makefiles. I had intended not to read files whose name contains $
characters, but had not written exactly that in the code.
Fixes PR 31213.
2005-09-09 11:05:00 +00:00
rillig
15b9d4d943 Bugfix: The subdirectories of the pkgsrc root directory are listed
correctly. With the -r option, the order of directories is now
depth-first.
2005-09-06 10:30:00 +00:00
rillig
c3115e6a7c Added a check for variables that are modified using "+=". As they are
mostly lists of something, their name should be a plural form. There are
many exceptions to this rule, mostly because of backwards compatibility.
2005-09-05 15:45:29 +00:00
rillig
ffc5bb8970 Replaced two instances of ad-hoc regular expressions with regex_varassign. 2005-09-05 13:27:36 +00:00
rillig
771539924d The continuation line indicator in regex_varassign is not needed
anymore, since assignments are checked on logical lines instead of
physical lines.
2005-09-05 13:06:20 +00:00
rillig
29776bf19e Use logical lines for the validation of Makefiles rather than physical
lines. Print line ranges instead of the starting line in diagnostics.
2005-09-04 21:06:05 +00:00
hubertf
cd5fa34ef1 Update url2pkg to 1.29. Change:
Patch by pancake <pancake@phreaker.net> that permits url2pkg to scan
bsd.sites.mk and recreate the MASTER_SITE string. He has tested a bit
against GNU, GNOME, SOURCEFORGE master sites, and seems to do the things
fine.
2005-09-04 20:02:55 +00:00
wiz
072a396460 regen (under 2.0) 2005-09-04 18:45:46 +00:00
wiz
1d897c6696 New sentence, new line. 2005-09-04 18:45:36 +00:00
rillig
71c7552b19 Changed the wording of the diagnostics message when a patch contains an
RCS tag. Suggested by hubertf.
2005-09-04 00:33:18 +00:00
rillig
4184bfe32f Made the code in get_logical_line easier. 2005-09-04 00:26:13 +00:00
rillig
188a8676f4 Updated the man pages. 2005-09-04 00:04:15 +00:00
rillig
c99410f597 When RCS Ids are detected in patches, don't suggest the -ko mode, but
ask the user to remove the offending line from the patch.
2005-09-03 22:23:46 +00:00
rillig
ab783f6d5a As the shell commands in Makefiles can be quite long, only the close
neighborhood of the tool name is printed when checking for direct use of
tools.
2005-09-03 11:53:15 +00:00
rillig
d646a03983 Added section comments and renamed some subroutines and variables. 2005-09-03 10:41:23 +00:00
rillig
b27cb882b9 Made the import clauses uniform. 2005-09-03 10:19:05 +00:00
rillig
b57ad50501 Removed the -Winternal flag and the checks for the ":=" operator. There had
been too many spurious warnings to be worth the effort.
2005-09-02 22:45:31 +00:00
rillig
5ba3f69424 Removed the (undocumented) -d/--debug option. 2005-09-02 12:39:42 +00:00
rillig
8e2103a120 The command line options are now defined in one table instead of two. 2005-09-02 12:10:16 +00:00
rillig
3b87a70054 In logical lines from Makefiles, white-space surrounding the
continuation is stripped from the string to make the output shorter.
2005-09-02 11:43:52 +00:00
rillig
093cc95760 In the diagnostics, "../../" path components that are not at the
beginning are stripped to make the output less verbose. The beginning is
needed to have a reference from which package the file is included, so
it is not stripped.
2005-09-02 10:42:18 +00:00
rillig
1e45f525ef The PKGDIR variable is extracted from the Makefile first, because other
variables like PATCHDIR and DISTINFO_FILE depend on it.
2005-09-02 10:11:24 +00:00
rillig
5deb0fcc3e When checking for direct use of tools, the Makefile lines are
interpreted as "logical lines", that is, lines ending in a backslash are
continued on the next line. This led to a false warning for
PKG_FAIL_REASON, which is suppressed.
2005-09-02 01:03:20 +00:00
rillig
1c2f92a2b4 Fixed a newly introduced bug that prevented multiple patches per file from
being detected.
2005-09-01 23:24:35 +00:00
rillig
772fe05470 Fixed the false warnings for lines in patches that start with "---". The
old parser had been a little too simple.
2005-09-01 22:45:16 +00:00
rillig
0753a70b6e Pkglint can check the pkgsrc root directory now. The checks are kept
minimal, as I use it mainly to check all the categories using the -r
flag.
2005-09-01 22:09:39 +00:00
rillig
ac22c61ffe Added an (undocumented) option -r/--recursive that checks all the packages
in a category Makefile that are not commented out.
2005-09-01 21:39:57 +00:00
hubertf
55592c0463 fix typo in comment, reported by Leonard Schmidt on tech-pkg@ 2005-09-01 18:09:26 +00:00
xtraeme
7d23f8d60d Update to 20050817.
Changes:

	* Patches to make this build on NetBSD -current
	* Various contribs plus colorization and fixes
2005-08-25 10:34:12 +00:00
rillig
6dc00852e4 Added a check for PLISTs: Man pages should be installed into man/, not
share/man/.
2005-08-25 07:27:23 +00:00
rillig
a5453ed697 Removed all references to the -B/contblank command line option. 2005-08-25 07:24:00 +00:00
rillig
06693d3602 Removed the -p flag from the usage message. It hasn't been implemented
for quite some time. I don't know when it had been removed, but
apparently no-one had used it anyway.
2005-08-24 17:32:43 +00:00
rillig
d6720c2287 Removed the check for contiguous blank lines completely. As long as
pkglint does not catch the important facts correctly it shouldn't be
nitpicking about white-space. I doubt that anyone has used the -B# and
the -Wwhitespace options at all, so there's no harm removing them.
2005-08-24 17:29:52 +00:00
rillig
3497c6f0f7 Added a -Wwhitespace flag that is off by default to control whether
multiple contiguous blank lines should result in warnings.
2005-08-24 16:50:13 +00:00
jmmv
27d05a3d93 Update to 1.22:
- Honour INSTALL_PACKAGES during the 'auto' target.  Found by jwise@.
2005-08-23 14:52:58 +00:00
rillig
5a878cee18 Sorted the regular expression constants alphabetically. Renamed two of them. 2005-08-22 13:51:40 +00:00
rillig
fb408275e2 Don't warn about unsorted PLIST entries that contain ${foo}. Only make
sure the others are sorted.
2005-08-22 05:12:01 +00:00