pkgsrc/pkgtools/pkglint/TODO

* fix false positive warnings
* ONLY_FOR_PLATFORM => NOT_FOR_PLATFORM
* -Wl,-rpath
* include "../../mk/bsd.prefs.mk" before using "+=".
* include "../../mk/bsd.prefs.mk" before any .if
* warn about the use of ${SED} in {pre,do,post}-install
* check for direct use of user names and group names
* Find a reliable, secure way to delete all CVS directories inside
  ${WRKSRC}. When that is finished, add a warning that using ${XARGS}
  is insecure.
* detect spelling errors (like CONFLICT => CONFLICTS)
* warn about the use of ${WRKDIR:=...}, as this construct should only
  be used with lists.
* mention the SUBST framework whenever ${SED} is used.
* check that Index: entries match the rest of the patch
* record with each substring its source location to be able to make
  even more precise diagnostics
Added TODO list for pkglint. 2005-04-30 13:50:42 +02:00			`* fix false positive warnings`
Checking for C/C++ comments is not necessary for Makefiles. No-one uses them. All pathnames are already interpreted as being relative to the package's directory. Added a reminder that ONLY_FOR_PLATFORM should be changed to NOT_FOR_PLATFORM where appropriate. 2005-08-06 10:00:14 +02:00			`* ONLY_FOR_PLATFORM => NOT_FOR_PLATFORM`
"-Wl,-rpath" should not be used directly. ${COMPILER_RPATH_FLAG} is better. 2005-10-13 22:17:45 +02:00			`* -Wl,-rpath`
include ../../mk/bsd.prefs.mk before using "+=". 2005-11-04 00:28:09 +01:00			`* include "../../mk/bsd.prefs.mk" before using "+=".`
+ include "../../mk/bsd.prefs.mk" before any .if. 2005-11-06 16:55:24 +01:00			`* include "../../mk/bsd.prefs.mk" before any .if`
* check Makefile directives * check _every_ line of the Makefiles * warn about the use of ${SED} in {pre,do,post}-install 2005-11-06 16:59:48 +01:00			`* warn about the use of ${SED} in {pre,do,post}-install`
diff: - check for magic numbers like 0755, 755, 644 + check for direct use of user names and group names 2005-11-08 22:00:33 +01:00			`* check for direct use of user names and group names`
* Find a reliable, secure way to delete all CVS directories inside ${WRKSRC}. When that is finished, add a warning that using ${XARGS} is insecure. 2005-11-10 13:19:58 +01:00			`* Find a reliable, secure way to delete all CVS directories inside`
			`${WRKSRC}. When that is finished, add a warning that using ${XARGS}`
			`is insecure.`
* detect spelling errors in variable names 2005-11-14 07:35:36 +01:00			`* detect spelling errors (like CONFLICT => CONFLICTS)`
* warn about the use of ${WRKDIR:=...}, as this construct should only be used with lists. 2005-11-14 14:07:05 +01:00			`* warn about the use of ${WRKDIR:=...}, as this construct should only`
			`be used with lists.`
* mention the SUBST framework whenever ${SED} is used. 2005-11-15 11:30:54 +01:00			`* mention the SUBST framework whenever ${SED} is used.`
Index: has sometimes different precedence in patch, avoid problems. 2005-12-30 17:27:13 +01:00			`* check that Index: entries match the rest of the patch`
* record with each substring its source location to be able to make even more precise diagnostics 2006-01-02 10:42:52 +01:00			`* record with each substring its source location to be able to make`
			`even more precise diagnostics`