Age | Commit message (Collapse) | Author | Files | Lines |
|
pkglint -Wall -F --only aligned --only indent -r
No manual corrections.
|
|
pkglint -Wall -r --autofix --only aligned
After running the above command, three packages had to be adjusted
manually:
In maelstrom-x11, pkglint didn't see that a commented variable
assignment should have been re-aligned as well.
In netmaze, there was a single line that didn't align well with all the
others, but pkglint decided to adjust the whole paragraph to this single
line. The result would have been ok, but not good.
In tscp, there was a paragraph where some of the lines were indented
one space more than the others. This rule is useful when one variable
name is very long compared to the others, but not in this case.
|
|
Therefore it is not necessary to add the :Q modifier to this variable
and several others.
pkglint -r -Wall --autofix --only :Q
|
|
Problems found locating distfiles:
Package blokus: missing distfile blokus-1.1.tar.gz
Package crossfire-client: missing distfile crossfire-client-images-1.11.0.tar.gz
Package dungeon: missing distfile dungeon-gdt-glk.tar.gz
Package gate88: missing distfile Gate88_Mar19_05.tar.gz
Package gnuchess4: missing distfile gnuchess-4.0.pl80.tar.gz
Package ioquake3-pk3: missing distfile quake3-latest-pk3s.zip
Package rollemup: missing distfile Rollemup.tar.gz
Package tscp: missing distfile tscp181.zip
Package xbat: missing distfile xev111.tar.gz
Package xgalaga: missing distfile xgalaga-2.0.34.tar.gz
Package xrick: missing distfile xrick-021212.tgz
Package xskat: missing distfile xskat-cards.tar.gz
Otherwise, existing SHA1 digests verified and found to be the same on
the machine holding the existing distfiles (morden). All existing
SHA1 digests retained for now as an audit trail.
|
|
|
|
* libXp was used by Xaw8, but it had been obsolated, and in pkgsrc,
x11/libXaw/buildlink3.mk had been switched to pick up Xaw7 by default.
* With x11/xorg-cf-files, libXp was offered with XawClientLibs,
but updated to 1.0.4, it was removed.
* And pkgsrc had been switched to use always xorg-cf-files and imake from pkgsrc,
so all platforms should not require libXp from libXaw with Imake.
Bump PKGREVISION.
|
|
|
|
|
|
|
|
source.
|
|
|
|
or USE_X11BASE set, but don't include mk/x11.buildlink3.mk directly or
via buildlink3.mks
- introduce BUILDLINK_PREFIX.libXpm as alias for BUILDLINK_PREFIX.xpm
in the !modular case
- fix some cases where the check for libX11 couldn't work at all by using
C++ for compilation without including the proper headers
Verified using a full X11_TYPE=xorg bulk build without additional
breakage. Discussed with salo@, wiz@ and send to packages@ for feedback.
|
|
|
|
I haven't changed the package to use setgid game and removed the
other-writeable, since such files should not be created under
${LOCALBASE}. Bump revision.
|
|
|
|
example MAKE_ENV+=FOO=${BAR} is changed to MAKE_ENV+=FOO=${BAR:Q}. Some
other changes are outlined in
http://mail-index.netbsd.org/tech-pkg/2005/12/02/0034.html
|
|
useful.
|
|
Probably fixes PR 29490.
|
|
|
|
Suggested by Roland Illig, ok'd by various.
|
|
|
|
Remove `-p' from mkdir arguments, it is already part of ${MKDIR}.
While here substitute a couple of ${PREFIX} by `%D' in
`@exec ${MKDIR} ...' lines and add a couple of missing `%D' in such lines too!
|
|
rmdir -> ${RMDIR}
rm -> ${RM} (${RM} added to PLIST_SUBST)
chmod -> ${CHMOD}
chown -> ${CHOWN}
|
|
chmod -> ${CHMOD}
|
|
|
|
|
|
+ move the patch digest/checksum values from files/patch-sum to distinfo
|
|
XXX need to teach pkglint to be more picky about this
|
|
|
|
If anywhere, it should be the value of MAINTAINER in the Makefile.
Some minor cleanup/reformatting while I'm here.
|
|
|
|
|
|
|
|
A game of breaking blocks
|