| Age | Commit message (Collapse) | Author | Files | Lines | |
|---|---|---|---|---|---|
| 2005-09-09 | This is pkglint-4.21. | rillig | 1 | -2/+2 | |
| 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 | Changed the regular expression for detecting valid .include lines in | rillig | 1 | -2/+2 | |
| 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-06 | Bugfix: The subdirectories of the pkgsrc root directory are listed | rillig | 1 | -7/+16 | |
| correctly. With the -r option, the order of directories is now depth-first. | |||||
| 2005-09-05 | Added a check for variables that are modified using "+=". As they are | rillig | 1 | -1/+75 | |
| 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 | Replaced two instances of ad-hoc regular expressions with regex_varassign. | rillig | 1 | -4/+4 | |
| 2005-09-05 | The continuation line indicator in regex_varassign is not needed | rillig | 1 | -4/+4 | |
| anymore, since assignments are checked on logical lines instead of physical lines. | |||||
| 2005-09-04 | Use logical lines for the validation of Makefiles rather than physical | rillig | 1 | -20/+43 | |
| lines. Print line ranges instead of the starting line in diagnostics. | |||||
| 2005-09-04 | regen (under 2.0) | wiz | 1 | -15/+15 | |
| 2005-09-04 | New sentence, new line. | wiz | 1 | -2/+3 | |
| 2005-09-04 | Changed the wording of the diagnostics message when a patch contains an | rillig | 1 | -5/+4 | |
| RCS tag. Suggested by hubertf. | |||||
| 2005-09-04 | Made the code in get_logical_line easier. | rillig | 1 | -3/+8 | |
| 2005-09-04 | Updated the man pages. | rillig | 2 | -36/+30 | |
| 2005-09-03 | When RCS Ids are detected in patches, don't suggest the -ko mode, but | rillig | 1 | -2/+2 | |
| ask the user to remove the offending line from the patch. | |||||
| 2005-09-03 | As the shell commands in Makefiles can be quite long, only the close | rillig | 1 | -8/+23 | |
| neighborhood of the tool name is printed when checking for direct use of tools. | |||||
| 2005-09-03 | Added section comments and renamed some subroutines and variables. | rillig | 1 | -28/+27 | |
| 2005-09-03 | Made the import clauses uniform. | rillig | 1 | -5/+11 | |
| 2005-09-02 | Removed the -Winternal flag and the checks for the ":=" operator. There had | rillig | 1 | -16/+1 | |
| been too many spurious warnings to be worth the effort. | |||||
| 2005-09-02 | Removed the (undocumented) -d/--debug option. | rillig | 1 | -16/+1 | |
| 2005-09-02 | The command line options are now defined in one table instead of two. | rillig | 1 | -49/+60 | |
| 2005-09-02 | In logical lines from Makefiles, white-space surrounding the | rillig | 1 | -8/+13 | |
| continuation is stripped from the string to make the output shorter. | |||||
| 2005-09-02 | In the diagnostics, "../../" path components that are not at the | rillig | 1 | -1/+5 | |
| 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 | The PKGDIR variable is extracted from the Makefile first, because other | rillig | 1 | -7/+6 | |
| variables like PATCHDIR and DISTINFO_FILE depend on it. | |||||
| 2005-09-02 | When checking for direct use of tools, the Makefile lines are | rillig | 1 | -8/+27 | |
| 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-01 | Fixed a newly introduced bug that prevented multiple patches per file from | rillig | 1 | -1/+3 | |
| being detected. | |||||
| 2005-09-01 | Fixed the false warnings for lines in patches that start with "---". The | rillig | 1 | -3/+10 | |
| old parser had been a little too simple. | |||||
| 2005-09-01 | Pkglint can check the pkgsrc root directory now. The checks are kept | rillig | 1 | -7/+59 | |
| minimal, as I use it mainly to check all the categories using the -r flag. | |||||
| 2005-09-01 | Added an (undocumented) option -r/--recursive that checks all the packages | rillig | 1 | -9/+14 | |
| in a category Makefile that are not commented out. | |||||
| 2005-08-25 | Added a check for PLISTs: Man pages should be installed into man/, not | rillig | 1 | -1/+4 | |
| share/man/. | |||||
| 2005-08-25 | Removed all references to the -B/contblank command line option. | rillig | 2 | -12/+4 | |
| 2005-08-24 | Removed the -p flag from the usage message. It hasn't been implemented | rillig | 1 | -2/+1 | |
| for quite some time. I don't know when it had been removed, but apparently no-one had used it anyway. | |||||
| 2005-08-24 | Removed the check for contiguous blank lines completely. As long as | rillig | 1 | -11/+1 | |
| 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 | Added a -Wwhitespace flag that is off by default to control whether | rillig | 1 | -2/+4 | |
| multiple contiguous blank lines should result in warnings. | |||||
| 2005-08-22 | Sorted the regular expression constants alphabetically. Renamed two of them. | rillig | 1 | -7/+7 | |
| 2005-08-22 | Don't warn about unsorted PLIST entries that contain ${foo}. Only make | rillig | 1 | -33/+17 | |
| sure the others are sorted. | |||||
| 2005-08-21 | Reduced the number of warnings for replacing ":=" with "=". Added a command | rillig | 1 | -3/+10 | |
| line flag -Winternal to enable checking of the pkgsrc infrastructure files. By default these files are not checked. | |||||
| 2005-08-21 | Buildlink files may use the := assignment operator for variable names that | rillig | 1 | -2/+2 | |
| contain "BUILDLINK". All other uses are still flagged. | |||||
| 2005-08-21 | Added a flag -g or --gcc-output-format that makes the messages parseable by | rillig | 1 | -29/+51 | |
| the Emacs compile.el program. Changed the keyword for errors from FATAL to ERROR. Sorted command line options alphabetically in the source code. | |||||
| 2005-08-21 | This is pkglint-4.20. Changes since 4.19 include: | rillig | 1 | -2/+2 | |
| - Very much code cleanup - Introduction of the --autofix flag for category Makefiles - Rewrite of the code to check the DISTNAME section in package Makefiles - Rewrite of the code to check category Makefiles | |||||
| 2005-08-21 | Generally warn about the use of the := operator in Makefiles. | rillig | 1 | -2/+10 | |
| 2005-08-21 | Rewrote the checking of the Makefile variables in the DISTNAME section. | rillig | 1 | -74/+67 | |
| This reduces the number of false warnings. | |||||
| 2005-08-20 | Fixed a syntax error (a string had not been enclosed in double quotes). | rillig | 1 | -2/+2 | |
| 2005-08-20 | The default values for some variables in checkfile_package_Makefile are | rillig | 1 | -24/+19 | |
| chosen more carefully to avoid false warnings. The local variables are sorted by categories. Unnecessary variables have been removed. | |||||
| 2005-08-19 | Added the command line option -CDESCR, for consistency. | rillig | 1 | -2/+4 | |
| 2005-08-19 | The --help message was wrong. The progress messages are not printed on | rillig | 1 | -2/+2 | |
| STDERR but on STDOUT. | |||||
| 2005-08-19 | Reduced two errors to warnings for the category Makefiles: The sort | rillig | 1 | -3/+3 | |
| order and subdirectories which are commented out without giving a reason. | |||||
| 2005-08-19 | Don't check the distinfo twice if it is defined as ./distinfo in the | rillig | 1 | -2/+2 | |
| package Makefile. This occurs for example in multimedia/gmplayer. | |||||
| 2005-08-17 | Changed the way how the direct use of tools is detected in Makefiles. | rillig | 1 | -14/+20 | |
| Now the valid sequences are deleted from the string. If after the deletion the string contains a tool name, a warning is printed. | |||||
| 2005-08-17 | Moved the checkdir_package subroutine directly before the main subroutine | rillig | 1 | -97/+86 | |
| to reduce the number of necessary forward declarations. Subsequently, removed those forward declarations. | |||||
| 2005-08-17 | Renamed check_{package,category} to checkdir_{package,category} to | rillig | 1 | -7/+7 | |
| distinguish them from the checkfile_* and checklines_* subroutines. | |||||
| 2005-08-17 | Fixed some bugs that appeared while checking the databases/ category | rillig | 1 | -3/+4 | |
| Makefile. | |||||
 |
index : pkgsrc | |
| [no description] | Dyson |
| summaryrefslogtreecommitdiff |