Fix insecure /tmp usage in sail.

Essentially concludes the merge into NetBSD of those changes in the
Linux port of the NetBSD games which it is fairly clear to me should
be so merged.

Add NetBSD slogans for fortune database.

Don't initialise curses until after the initial non-curses setup (the
choice of scenario uses puts() rather than curses functions).

need an ${.OBJDIR} now

More .mk file fallout.  *sigh*

Get the build ordering right for dependall.

More .mk file fallout.  *sigh*

Add manpage wargames(6), contributed by Joey Hess <joeyh@kitenet.net>.

No special formatting used for the movie name for now, following
discussion on tech-userlevel suggesting that at present there's
nothing really appropriate.

Nuke `extern int errno;' in code we compile with -Wstrict-prototypes.  We get
the correct definition from errno.h.

Clean up error reporting in tetris.

Fix high score table formatting bug.

Bug report and fix from Malcolm Parsons <malcolm@bits.bris.ac.uk>.

Improve snscore error handling.

Report scores as auto-bot or not correctly.

Bug report and fix from Malcolm Parsons <malcolm@bits.bris.ac.uk>.

Ensure correct prompt asking about another game is given at the end of
each game.

Bug report and fix from Malcolm Parsons <malcolm@bits.bris.ac.uk>.

Typo fix in unknown option message (from Malcolm Parsons
<malcolm@bits.bris.ac.uk>).

Remove definition of __cputchar.

Define and use MAXRANDOM instead of LONG_MAX.

Remove unnecessary casts, and add necessary ones for printf arguments.

Change __GCC__ to __GNUC__.

Remove obsolete NO_VOID conditional.

Use err(1, NULL) and warn(NULL) for `out of memory' error messages.

Declare variables as extern in headers rather than using linker commons.

Don't use <err.h>.  Use ANSI prototypes (no dependance on the __P()
macro).  Add new fwrite_be_offt() function which writes out a big-endian
64bit number regards of the size of off_t on the host machine.

Also fix ``bug'' in previous - even though the off_t pointers were
stored big-endian, the individual card lenghs were in host-order.  The
cards.pck files are now identical when make on a big- or little-endian
machine.

Now builds a correct cards.pck file on non-NetBSD machines as well
- tested on Ultrix and Solaris (but Solaris needs -lsocket to get
htonl()).

Don't use any NetBSD specific features (<err.h> and friends mainly
- replaiced with local die() and dieperror() routines).  Use ANSI
prototypes (no dependance on the __P() macro).  Add new fwrite_be_offt()
function which writes out a big-endian 64bit number regards of the size
of off_t on the host machine.  Remove unused unctrl() function.

Now builds correct fortune .dat files on non-NetBSD machines as well
- tested on Ultrix and Solaris (but Solaris needs -lsocket to get
htonl()).

No need to include <sys/endian.h> here - <sys/types.h> includes
<machine/endian.h> which DTRT.

Use separate definitions and extern declarations of variables rather
than linker commons.

Fix capital of Monaco.  It should be Monaco rather than Monte Carlo;
checked with CIA World Factbook.

Problem reported by <capo@writeme.com>.

Make quiz use POSIX.2 pager handling.

Print a space character on the tape between each command line argument.

Bug fix from OpenBSD.

Removed someone's personal phone number, including area code, from a "if you
want a good time, call ###-###-####" fortune entry.

Fix a gcc -W warning (empty body in an else-statement).

If the dictionary for boggle does not include words beginning with
every letter, add appropriate index lines.  Problem reported by Mike
Castle <dalgoda@ix.netcom.com> against the Linux port of the NetBSD games
collection.

Correct wargames quote, and while we are here be a little more selective
when checking if a game is valid (eg: '../../bin/csh')

Do not match non-literals (REs).

A slightly different patch was submitted in PR 8388 by
Antti Kantee <pooka@iki.fi>

Since our gcc doesn't warn about NULL format strings anymore, we can
fix the incorrect err(1, "%s", "") et al.
Closes PR bin/7592 by cgd.

fix handling of '-t' flag, and pause for a little bit after drawing happysnake

correct description of representation of snake

support arrow keys.

update after change to return value of tputs() third argument

Fix error in last namespace change (only relevant #ifdef DEBUG).

Add more files to FILES section of cribbage.6.

Fix uses of namespaces reserved by ISO C or POSIX.1.

Use macro NAMLEN rather than dirent->d_namlen, for easier portability.

Use the function usage().

Don't include terminal . in argument of err().

Don't include <nlist.h>.

Remove generated file "limerick-o" in "clean" target.

Cleanup and warning fixes for phantasia/setup.c.

Allow dm to ban games playing 11pm-midnight.

From OpenBSD.

Define variables as BOOLEAN, not bool, where the extern declarations
already use BOOLEAN.

Further warning fix for mkdict.

Add alternative style for commands at end of game (#ifdef NEW_STYLE).

Warning cleanup for boggle's mkdict and mkindex.

Fix -Wsign-compare warnings.

Use strncpy() rather than strcpy() with fixed size buffer.

Add missing initialisers.

Add FALLTHROUGH comment.

Give sensible error message in a shouldn't happen case.

Make word list for hangman selectable at run time.

Convert tetris to use <err.h> functions.

Fix some uses of NULL as non-pointer and 0 for null pointer.

Use `extern' for header variable declarations in trek(6).

Change quiz `pres' data file to include vice-presidents, and fix some
minor errors.

From OpenBSD.

Update pom to the third edition of Duffett-Smith's book, and clean it
up somewhat.  Change time specification format to similar to that used
by date(1) - taking a command line argument in seconds since the Epoch
is silly.  Date parsing based on code from date(1).

Based on work by Paul Janzen for OpenBSD.

Declare variables in extern.h as extern, and add non-extern
declarations to globals.c.

Use time() instead of gettimeofday().

Convert snake to use curses, and generally clean it up somewhat.

Includes bugfixes from or inspired by OpenBSD.

Make cribbage use the POSIX.2 pager behaviour.

Fix buffer overrun in rogue.

Fix uses of names of library functions for other purposes.

A couple more checks for file descriptor < 3 missed earlier.

Remove rogue's old and bit-rotten cut-down version of curses.

Split limericks into "normal" and "offensive" ones.
Fixes PR 6202 by Matthias Grutzeck <grut@teco.edu>.

These fortunes conatain offensive language.
These were originally unamerican.{fake,real}
Part of the work on PR 6202.

Moved to unamerican.real-o

Moved to unamerican.fake-o

Security improvements for games (largely from or inspired by OpenBSD).

Games which run setgid from dm, but don't need to, should drop their
privileges at startup.

Games which have a scorefile should open it at startup, then drop all
privileges leaving just the open writable file descriptor.  If the
game can invoke subprocesses, this should be made close-on-exec.

Games with scorefiles should make sure they do not get a file
descriptor < 3.  (Otherwise, they could get confused and corrupt the
scorefile when using stdin, stdout or stderr.)

Some old setuid revokes from the days of setuid games change into gid
revokes.

Use STD*_FILENO.

Check for write errors in auxiliary programs used during build of games.

Compare getchar() against EOF, not -1.

Fix glitch in const changes (bin/8365 from John.P.Darrow@wheaton.edu).

Include <time.h> in various places in the games where time() or time_t
are used.

Check for failure of malloc() and calloc() at various places in the games.

Remove extra blank line in copyright.

Use the symbolic names `SEEK_SET' and `O_RDONLY' where appropriate in
the games.

Add `__noreturn__' and `__unused__' attributes where appropriate to
the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Most noreturn attributes were previously added in bin/6144, with some
others that were missed then in bin/8082.  Previous `unused'
attributes were covered in bin/6557, bin/8058 and other PRs (all these
PRs have already been handled and closed).

Add use of `const' where appropriate to the games.

This merges in all such remaining changes from the Linux port of the
NetBSD games, except in hunt (where substantial changes from OpenBSD
need to be looked at).

Some such changes were previously covered in PRs bin/6041, bin/6146,
bin/6148, bin/6150, bin/6151, bin/6580, bin/6660, bin/7993, bin/7994,
bin/8039, bin/8057 and bin/8093.

Removed unused array Movenames[].

Convert to something resembling KNF.

Instead of writing out a structure that contains pointers as the header
of the card decks file, just write out the number of cards for each
deck.  Also use "off_t" for offsets into the file (that are stored after
the number of cards) instead of "long".
/usr/share/games/cards.pck is now MI.

Use new endian-specific conversion macros - 64-bit off_t's are now
the stored the same regardess of the byte order of the generating
host.

Note in the strfile(8) man page that all fields are big-endian, not
in network byte order.

Don't chown installed files or directories if UNPRIVILEGED is defined.

"make build" should now work as a non-root user (tested on Alpha).
mtree spits out lots of warnings during "make distrib-dirs", but
these are non-fatal.

Use ${INSTPRIV} with ${INSTALL}.

Make phrase to enter password again clearer

In name section, use .Dq macro rather than writing double quote directly.

Spell "privilege" correctly (correct spelling from Jonathan Stone).

Adapt to change of "termcap.h".

Adapt to change in "termcap.h".

Adapt to change of "termcap.h".

Fix the part of pr8201 which is IMHO a bug: the error message now goes
to stderr.

The existing error message ("I don't know what xxx means") is
unchanged, as it is stylisticly in keeping with the light-hearted
nature of the program, and is also more likely to be accurate than the
"no such acronym" error message proposed in 8201.