Enable FANCY mode for debugging purposes:
- To enable stand still mode (robot is not moving but waiting in place),
specify a scorefile on the command line called <directory>/stand_still
- To enable pattern roll mode (robot is changing which move to look first)
specify a scorefile on the command line called <directory>/pattern_roll

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.

Add -A for automatic playing mode.
Make the score file MI

Simply include -lcurses instead of -lcurses -ltermcap

Use -ltermcap instead of -ltermlib

don't clobber /var/games/robots_roll on install; from
NAKAJIMA Yoshihiro in PR bin/4657.

We need a FILESMODE=660 for dm which is now setgid games.

install games that need it setgid.

use CPPFLAGS instead of CFLAGS

branches: 1.8.2;
WARNSify

- Makefile cleanups

Use ${INSTALL}.

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

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

Remove clause 3 and 4 from TNF licenses

branches: 1.7.24;
describe the algorithm better.

Remove uses of __P.

Be more verbose about teleports, and fix a minor problem in autobot mode.

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.

Fixi uninit vars warnings.

Add missing prototypes

Add -A for automatic playing mode.
Make the score file MI

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

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

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

Fix -Wsign-compare warnings.

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.

Add -A for automatic playing mode.
Make the score file MI

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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

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

WARNSify

Use POSIX tty semantics.

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

fix various typos in comments and log messages, mainly s/hight/high/
s/hight/height/ and removing double m in time(ing).

branches: 1.12.58;
Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

A bit more whitespace.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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

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

Be more verbose about teleports, and fix a minor problem in autobot mode.

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.

WARNSify

Remove calls to flushok(). That function doesn't really do anything in
our curses implementation, and it doesn't even exist in others...

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Enable FANCY mode for debugging purposes:
- To enable stand still mode (robot is not moving but waiting in place),
specify a scorefile on the command line called <directory>/stand_still
- To enable pattern roll mode (robot is changing which move to look first)
specify a scorefile on the command line called <directory>/pattern_roll

robots: Use arc4random_uniform for better uniform distribution

sprinkle static

don't use char values for functions that can return -1; chars are not always
signed.

Use getopt instead of hand-rolled options code. Document all the arguments
and options. Don't allow the previously undocumented method to change the
maximum number of scores kept per user to be used on the system-wide high
score file. Sort options list in the man page. Bump its date.

Use random() instead of rand(), and seed with time instead of pid.

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

A bit more whitespace.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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.22.6;
convert __attribute__s to applicable cdefs.h macros

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

Change deprecated curses function to new equivalent.

fix nested externs

Use setgid(), not setregid().

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.

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

branches: 1.11.2;
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 `__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.

Add -A for automatic playing mode.
Make the score file MI

deprecate bzero, rindex, etc. use symbolic constants in open()

WARNSify

clean up import

Really fix exit code.

Rewrite grody exit code and eliminate use of _putchar().

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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

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

Add -A for automatic playing mode.
Make the score file MI

deprecate bzero, rindex, etc. use symbolic constants in open()

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

sprinkle static

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

- sleep a little in autobot mode so that we can see what is happening.
- use refresh() instead of wrefresh()

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

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

Be more verbose about teleports, and fix a minor problem in autobot mode.

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.

Add -A for automatic playing mode.
Make the score file MI

Fix for compiling with -DFANCY, as per PR 5832 by Joseph Myers <jsm28@cam.ac.uk>

WARNSify

Remove calls to flushok(). That function doesn't really do anything in
our curses implementation, and it doesn't even exist in others...

clean up import

Add RCS identifiers.

Make CTRL() ANSI-compliant.

branches: 1.1.1;
Initial revision

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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

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

comment or delete text after CPP directives.

Add -A for automatic playing mode.
Make the score file MI

WARNSify

clean up import

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

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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

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

comment or delete text after CPP directives.

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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

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

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.

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

robots: Use arc4random_uniform for better uniform distribution

sprinkle static

Use random() instead of rand(), and seed with time instead of pid.

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

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

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

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision

Use getopt instead of hand-rolled options code. Document all the arguments
and options. Don't allow the previously undocumented method to change the
maximum number of scores kept per user to be used on the system-wide high
score file. Sort options list in the man page. Bump its date.

\@ is not a valid escape sequence.

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

New sentences begin on new lines.
Patch from Richard Elz, slightly improved by yours truly.

Generate <>& symbolically. I'm avoiding .../dist/... directories for now.

Whitespace and/or punctuation fixes.

Add -A for automatic playing mode.
Make the score file MI

WARNSify

alternate -> alternative, per PR 2643

clean up import

Update to -mandoc macros.

Add RCS indentifiers.

branches: 1.1.1;
Initial revision

sprinkle static

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

Remove uses of __P.

Include <sys/endian.h>.

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

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

Be more verbose about teleports, and fix a minor problem in autobot mode.

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

branches: 1.13.2;
Fix -Wsign-compare warnings.

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.

Add missing prototypes

Add -A for automatic playing mode.
Make the score file MI

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

deprecate bzero, rindex, etc. use symbolic constants in open()

WARNSify

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

Add RCS identifiers.

Make CTRL() ANSI-compliant.

branches: 1.1.1;
Initial revision

games: Do not assume <sys/endian.h> is transcluded from <sys/types.h>

prompted by PR pkg/59839

properly terminate after using strncpy().

sprinkle static

don't ignore errors from read(); found by lint

Assorted minor cleanups, no functional change:
- u_int* -> uint*
- don't make private typedefs of system structures
- use curses TRUE and FALSE only with curses booleans, otherwise
true and false;
- includes cleanup
- group globals in extern.c by functionality

Object file diffs inspected.

Whitespace.

ANSIfy. Use __dead. Object diffs checked.

If getpwuid() returns null, its not usually a good idea to dereference
that to try to assign a default value. Just copy the default value into
the final destination.
Addresses Coverty CID 925

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.

Cast pointer diffs to long for printing with %ld.

Fix arguments to *printw().

Don't insert leading 0s in the score display. (HI CHRISTOS!)

Report scores as auto-bot or not correctly.

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

Fix -Wsign-compare warnings.

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 the symbolic names `SEEK_SET' and `O_RDONLY' where appropriate in
the games.

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.

Add missing prototypes

Add -A for automatic playing mode.
Make the score file MI

deprecate bzero, rindex, etc. use symbolic constants in open()

WARNSify

clean up import

Add RCS identifiers.

branches: 1.1.1;
Initial revision