Age | Commit message (Collapse) | Author | Files | Lines |
|
filenames now.
- MD5 checksums are marked as deprecated.
- Warnings from distinfo files now appear correctly sorted.
|
|
wrong ``false'' with $opt_warn_varorder.
- Fixed the detection of $seen_Makefile_common.
|
|
that ends in :Q.
- When NO_CONFIGURE ist defined, REPLACE_PERL does not work.
|
|
- Fixed GCC 4.x build warning (which would have resulted in an error
with future GCC versions).
- Depend on newer version of "rpm" package to fix build under Mac OS X.
|
|
"extract" script for extraction. Many cases where a custom EXTRACT_CMD
simply copied the distfile into the work directory are no longer
needed. The extract script also hides differences between pax and
tar behind a common command-line interface, so we no longer need code
that's conditional on whether EXTRACT_USING is tar or pax.
|
|
Changes since 4.53:
- Added about 50 new variables to be type-checked.
- For unknown variables, the type is guessed from the variable name.
- In patch files that patch Makefiles, added lines are checked for
absolute pathnames.
- Replaced the -Wdebug option with --debug, so that the debugging warnings
don't show up when using -Wall.
- Many minor improvements.
|
|
latter has been removed.
- Added -Wvarorder (default: disabled) to check the ordering of variables
in simple packages.
- Using ${WRKSRC}/.. is only considered a warning, not an error, because
there are packages that work nevertheless. Using this construct remains
discouraged.
- Replaced most of the TODOs in the source code with code that prints
a warning in --debug mode.
- The characters '.' and '+' are allowed in BuildlinkPackages.
- Variables *_FLAGS and *_ARGS that have their type guessed based on the
name don't have to be assigned using the += operator.
- Variables that are assigned using the != operator are not checked for
type compliance.
- README files are not marked as unknown when they appear in a package
directory.
|
|
|
|
|
|
a few weeks ago when I was doing some builds using modular-xorg.
(Now that new Xorg is in, maybe there are some others to add?)
Update version.
|
|
It has come to the point where almost any further improvement means to
adopt a real recursive parser.
|
|
../../bsd.prefs.mk is included. (It is only shown in -Wextra mode
because it is still quite unreliable.)
|
|
Fix incorrect quotation in test
|
|
precision in checkfile_patch for the warning for context diffs.
- The --source option produces more condensed output than before.
- The diagnostics with character-wise precision are colored using
ANSI/VT100 color sequences. Support for other output formats will be
added later.
|
|
other directories it is undefined. This avoids some false warnings when
checking files in pkgsrc/mk/.
|
|
|
|
with the change from December 2005, when using this feature, care should
be taken for variables that are pre-quoted.
- In patch files that patch Makefiles, added lines are checked for
absolute pathnames.
|
|
|
|
type List of Pathmask.
|
|
- Added some variable types.
- For unknown variables, the type is guessed from the variable name.
- Added more checks for relative pathnames.
- Improved the shell parser once more.
|
|
|
|
easily overflow the argument space e.g. on DragonFly. Replace this
statement with "ls -t | grep '\.tgz$'" to remove this constraint.
|
|
When using non-native X11, xpkgwedge could be installed before imake
and host.def en up with just the xpkgwedge.def include. This means
that ProjectRoot as set by xorg's host.def is not used and wrong
path names are used. This run time dependency is the easiest way to
ensure proper order, but isn't perfect. Just providing a fixed host.def
and a secondary include file for Lesstif to modify breaks with
USE_XPKGWEDGE=NO, so keep the old approach.
Discussed with jlam@
|
|
Changes since 4.52:
- New warning categories "debug", "space", "quoting" that can be selected
independently.
- Improved the parsers for shell commands and shell words.
- Added warning for using $(...) in shell programs, as the Solaris /bin/sh
doesn't support it.
- Added some warnings about quoting.
- Some other minor changes.
|
|
|
|
obsolete synonym for the pipe operator "|", so you must protect it with
quotes''. (Kernighan/Pike, The UNIX Programming Environment, 1984.)
- $@ is another well-known shell variable.
- All substitution arguments to ${SED} and ${PAX} should be quoted whether
they contain special characters or not.
|
|
occurences of a $ in double quoted strings result in a warning.
|
|
are only shown in -Wdebug mode, as the diagnostic messages are too
cryptic.
- Added detection for the $(...) replacement for backticks, as the Solaris
/bin/sh cannot handle them.
|
|
worth a warning, not an error. There are worse things at the moment that
need to be fixed.
|
|
be enabled independently. They are all disabled by default.
- A note is emitted if variable definitions are aligned using space instead
of tabs.
|
|
precision of the diagnostics. It is currently unused.
|
|
|
|
after parsing the command line options of ${INSTALL}.
- Don't warn for CFLAGS that reference other make variables.
|
|
in about half the cases -- added an additional parameter.
|
|
|
|
upcoming changes.
|
|
X11 packages work on amd64 linux boxes. From Benjamin Shi.
|
|
Changes since 4.51:
- The occurence of a symlink other than work* inside the pkgsrc tree is
not an error, just a warning.
- SUBST_MESSAGE should not be quoted.
- If -Wextra is enabled, all make variables that are used in shell commands
are checked for proper quoting.
- Improved the parser for shell commands.
- Some other minor changes.
|
|
|
|
${PREFIX:Q}/foo.
- The semicolon after the "for" loop in the shell does not mean that some
exitcode is ignored, so don't print warnings for it.
|
|
|
|
- It is an error if the COMMENT has the default value from url2pkg.
|
|
|
|
code now needs less lines than before. For unknown state/shellword
combinations, warnings are printed (currently none).
|
|
positive warning in mk/subst.mk.)
- Improved debugging support for parsing shell commands by printing the
current state and word in -vv mode.
|
|
Use MaxPathSize instead of PATH_MAX.
|
|
direction: it had been quoted too much in the past).
|
|
remove the quoting from the definitions in the Makefiles.
- When checking with -Wall, print warnings about possibly misquoted
variables in the various states of the shell word parser.
|
|
Fix libkver support for pkg_comp's build, auto and install targets,
Reported broken and fix tested by salo@.
Also ensure libkver is built, if needed, even before digest.
Approved by jmmv@
|
|
subroutine, as it has no code that is specific to patch files.
|