51 lines
2.5 KiB
Text
51 lines
2.5 KiB
Text
$NetBSD: TODO,v 1.68 2008/10/09 16:01:51 rillig Exp $
|
|
|
|
Please add your own entries at the bottom of this file. If possible,
|
|
include the name of an example package where a warning should occur.
|
|
|
|
* fix false positive warnings
|
|
* warn about the use of ${WRKDIR:=...}, as this construct should only
|
|
be used with lists.
|
|
* record with each substring its source location to be able to make
|
|
even more precise diagnostics
|
|
* Many of the checking routines need some context to provide better
|
|
warnings. For example, checkline_foo() may need to know what has
|
|
happened in the file before that line.
|
|
* Add checks for binary packages. See Debian/lintian for ideas.
|
|
* Of the user-defined variables, some may be used at load-time and some
|
|
don't. Find out how pkglint can distinguish them.
|
|
* Make sure that no variable is modified at load-time after it has been
|
|
used once. This should at least flag BUILD_DEFS in bsd.pkg.mk.
|
|
* Implement unescape_shellword, which is required for the type check for
|
|
SedCommand.
|
|
* Invent an annotation scheme for files that intentionally define
|
|
variables for use in other files.
|
|
* ${MACHINE_ARCH}-${LOWER_OPSYS}elf in PLISTs etc. is a NetBSD config.guess
|
|
problem ==> use of ${APPEND_ELF}
|
|
* Packages including lang/python/extension.mk must follow the Python version
|
|
scheme. Enforcing PYPKGPREFIX for those is most likely a good idea.
|
|
* Check for parallel files/dirs whose names differ only in case.
|
|
* Check for license files that are completely unused.
|
|
* If a dependency depends on an option (in options.mk), it should also
|
|
depend on the same option in the buildlink3.mk file.
|
|
* Complain about ${PKGSRC_COMPILER} == "sunpro", which should be
|
|
!empty(PKGSRC_COMPILER:Msunpro).
|
|
* If USE_TOOLS has autoconf213, and the package does stuff like
|
|
cd ${WRKSRC} && autoconf, then an incorrect warning is issued.
|
|
* LOCALBASE should not be used in normal Makefiles
|
|
|
|
%
|
|
|
|
mail/mail-notification failed for Thomas, apparently because of an
|
|
unexpressed dependency on hicolor-icon-theme. I don't know if it's
|
|
possible to add a way to notice that the PLIST contains hicolor files
|
|
but that there's no dependency, or perhaps a list of mappings of sets of
|
|
PLIST regexes to sets of required packages, or something.
|
|
|
|
%
|
|
|
|
* don't complain about "procedure calls", like for pkg-build-options in
|
|
the various buildlink3.mk files.
|
|
* To push DESTDIR support forward, it would be useful it pkglint warned
|
|
like this if PKG_DESTDIR_SUPPORT is not defined:
|
|
"WARN: Please consider adding DESTDIR support to the package."
|