The custom C++ preprocessor thingy has been choking for a while on
clang; recently gcc's jointed the party. Work around this by building
it with -g instead of -O2. (For me at least with gcc, while -O2
produces a tool that gives spurious syntax errors, plain -O gives one
that loops forever instead, which isn't an improvement.)
Maybe someday someone will bother to find out where the undefined
behavior is.
Also build the actual game with -Wno-return-type to silence complaints
about dead functions not declared as dead and similar, and while here
adjust things so it still installs correctly regardless of umask.
- roll in patch-xfile.hc
- fix build with the solaris compiler
- use snprintf instead of sprintf (mostly)
- tidy a few things
There is a problem between this thing's precompiler and clang such
that clang builds fail with spurious errors about unterminated string
constants. I don't see what's wrong.
Also, in the course of doing the snprintf changes I discovered that
this depends on both csh and netpbm existing at runtime, so add the
necessary bits.
PKGREVISION -> 4.
Fix the fix previously in patch-ah (for platforms where char is
unsigned by default) to not generate spurious EOF indications.
Improve the fixes previously in patch-a[bcdef] to put the game's state
file in $HOME/.craft instead of the current directory.
Use a consistent method to set $PREFIX instead of a mixture of
compiler flags, patching, and SUBST.
Use pkg-config to find libX11, and clean up some pkglint.
- if we're going to supply the build makefile, it shouldn't require gmake.
- don't install executable html or gif files
- add patch comments
- const correctness for string constants, as demanded by gcc 4.5
- avoid needing -lcompat
- fix a y2038 issue
PKGREVISION -> 2
