clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

When using -lcurses, you also need -lterminfo.
This fixes the build for sun2, and also builds with LDSTATIC=-static,
since archive libraries don't record inter-library dependencies.

Simply include -lcurses instead of -lcurses -ltermcap

install games that need it setgid.

branches: 1.5.4;
clean up import

specify man pages the new way.

Add RCS identifiers.

got rid of unnecessary -lcompat and associated dependency

branches: 1.1.1;
Initial revision

Split struct betinfo into its own header file so it can be shared
between canfield and cfscores, instead of copy-pasted.

New sentence, new line.

Adjust SYNOPSIS for cfscores to better match reality. Note in BUGS
that the score file isn't portable. Bump date (first time since 1993,
and first in 2010...)

Use standard AUTHORS section header. From YOMURA Masanori in private mail
Sort sections if necessary. Use more/better markup.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

Lots of minor fixes resulting from reading these man pages in detail.

Whitespace and/or punctuation fixes.

Fix .Bl/.El imbalance.

WARNSify

clean up import

Update to -mandoc macros

Add RCS indentifiers.

branches: 1.1.1;
Initial revision

games: remove trailing whitespace in *.c and *.h

Use ssize_t for read() results. Don't use int for lseek() results.
Call srandom() with time(), not getpid().

Use uid_t to hold getuid() results, and don't check getuid for failure
as it isn't allowed to fail.

branches: 1.28.4; 1.28.10;
Don't use strings from arrays as format string.

Split struct betinfo into its own header file so it can be shared
between canfield and cfscores, instead of copy-pasted.

sprinkle static

if initscr() fails, exit with a message rather than crash in the
next curses call

Remove the \n and tabs from the __COPYRIGHT() strings.

branches: 1.23.6;
convert __attribute__s to applicable cdefs.h macros

branches: 1.22.10;
Fix typo in the instructions.

KNF and WARNS=3

Add (unsigned char) cast to ctype functions

Remove uses of __P.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

Use setgid(), not setregid().

Fix arguments to *printw().

branches: 1.15.6;
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.

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

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.

mark non-returning functions (PR#6144 by Joseph Myers <jsm28@cam.ac.uk>)

fix main() prototype, as per PR#5867

Use symbolic constants for seek(), open(). Fixed as per PR 6057 by
Joseph Myers <jsm@octomino.demon.co.uk>.

WARNSify

Clear the last char in input buffer when an erase or kill character is
read. This prevents rubbed-out commands from being accepted. (Fix
from PR #862)

Various changes to make games compile w/o warnings on the alpha:
Include appropriate includes, delete bogus function declarations,
change sizes of variables and casts.

clean up import

change <sys/signal.h> to <signal.h>

Changes for new Curses.

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

some makefile de-linting

MKfoo=no -> NOfoo=

convert from NOxxx= to MKxxx=no.
include <bsd.own.mk> if testing a MKxxx variable.

use CPPFLAGS instead of CFLAGS

branches: 1.3.4;
clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Use ssize_t for read() results. Don't use int for lseek() results.

branches: 1.21.6; 1.21.12;
Christos points out that usage should not use errx, but should use
getprogname.

Split struct betinfo into its own header file so it can be shared
between canfield and cfscores, instead of copy-pasted.

Use NULL instead of 0 for pointer tests. Remove an unnecessary cast.

Send error messages to stderr. Use errx/warnx, not printf.

Correct usage message.

sprinkle static

Remove the \n and tabs from the __COPYRIGHT() strings.

branches: 1.14.6;
Work around gcc signedness warning: if you check for uid_t < 0, gcc warns
because it's unsigned, but I don't really want to rely on uid_t being
unsigned on every platform this code might propagate to.

branches: 1.13.10;
KNF and WARNS=3

Remove uses of __P.

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22269, verified by myself.

More include additions for exit, abs, strcmp, etc.

Use setgid(), not setregid().

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.

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.

Use symbolic constants for seek(), open(). Fixed as per PR 6057 by
Joseph Myers <jsm@octomino.demon.co.uk>.

use err/warn instead of perror

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision