Rework how MAKEVERBOSE operates:

* Don't bother prefixing commands with a line of ${_MKCMD}\
and instead rely upon "make -s". This is less intrusive on
all the Makefiles than the former. Idea from David Laight.

* Rename the variables use to print messages. The scheme now is:
_MKMSG_FOO Run _MKMSG 'foo'
_MKTARGET_FOO Run _MKMSG_FOO ${.TARGET}
From discussion with Alistair Crooks.

rework to use the newer _MKMSGCREATE (et al) macros

Support MAKEVERBOSE. (Some host tools still generate verbose runtime messages)

use NETBSDSRCDIR as appropriate

There is no need to explcitly use ${.OBJDIR}; it's implied.

${PRINTOBJDIR}ify

More .mk file fallout. *sigh*

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

make objdir-aware

don't install in /usr/share if NOSHARE is defined

Use bsd.subdir.mk as appropriate.

Add .MAKE to the .OBJDIR hacks, so they work with `make -n'.

PR/3557: Dave Huang: use web2 not web2a

Make sure that the dictionary and indexing programs are made.

- Don't depend on installed ${DESTDIR}/usr/share/dict/words; pick up the
dictionary from the sources.
- Don't depend on the locally built indexing programs to build the dictionaries.

- Makefile cleanups

use words from $DESTDIR.

Remove read-only files before trying to rewrite them. From Greg Hudson.

add/fix id tags

remove extra tab

Remove hard-coded object directory.

branches: 1.1.1;
Initial revision

add/fix id tags

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.

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

don't install in /usr/share if NOSHARE is defined

Simply include -lcurses instead of -lcurses -ltermcap

Use -ltermcap instead of -ltermlib

- Makefile cleanups

Use ${INSTALL}.

add/fix id tags

specify man pages the new way.

branches: 1.1.1;
Initial revision

use size_t instead of int for a length

Factor out some common code; pass -Wstrict-overflow.

branches: 1.27.2; 1.27.8;
Use __dead and __printflike instead of __attribute__.

Fix a few of the games to use getprogname() in their usage() functions.

Suggested by Joerg.

XXX There are probably many more that have their progname hard-coded, but
XXX I'm not up for a wholesale roto-till at the moment.

Since this game is installed as 'boggle', fix the usage() text to match.

sprinkle static and prune some dead code

Rename internal getline() function to get_line() so it does
conflict with the soon to be added getline(3) libc function.

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

branches: 1.21.18;
PR/34175: Zafer Aydogan: touch the window to cause a refresh after help
Also use CTRL instead of \xxx

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().

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

Fix -Wsign-compare warnings.

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

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.

Drop setgid privs early. Via OpenBSD, reported in PR 5970 by
Joseph Myers <jsm28@cam.ac.uk>

From PR 7988 by Joseph Myers <jsm28@cam.ac.uk>: use the symbolic
name SEEK_SET for values which end up as the third argument of
fseek(), rather than a hard-coded 0.

Add braces to make the new egcs happy.

minor KNFify

WARNSify

Sync to 4.4BSD-Lite2

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.

add/fix id tags

make the code agree with documentation and usage() for final two
options.

Don't try to process args if they aren't there.

branches: 1.1.1;
Initial revision

Double the threshold for crashing out with "too many words".

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

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

add/fix id tags

branches: 1.1.1;
Initial revision

PR# 4305: [dM] boggle uses very incomplete wordlist.

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.

Fix macro order, and quote question mark.

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.

mdoc'ify, and clean up in the process.

add/fix id tags

branches: 1.1.1;
Initial revision

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

sprinkle static and prune some dead code

Rename internal getline() function to get_line() so it does
conflict with the soon to be added getline(3) libc function.

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.

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

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>)

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.

add/fix id tags

branches: 1.1.1;
Initial revision

NULL does not need a cast

KNF and WARNS=3

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

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

fix nested extern

WARNSify

add/fix id tags

branches: 1.1.1;
Initial revision

boggle(6), from 44lite

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

NULL does not need a cast

sprinkle static and prune some dead code

Rename internal getline() function to get_line() so it does
conflict with the soon to be added getline(3) libc function.

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

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

branches: 1.16.8;
PR/34175: Zafer Aydogan: touch the window to cause a refresh after help
Also use CTRL instead of \xxx

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.

fix nested extern

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.

time_t != long; cast it.

minor KNFify

WARNSify

Use tcflush().

Use POSIX signal and tty semantics.

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.

add/fix id tags

branches: 1.1.1;
Initial revision

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

fix unused variable warnings

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

Fix doubled 'the' in comment.

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.

minor KNFify

WARNSify

add/fix id tags

branches: 1.1.1;
Initial revision

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.

Slight redux.

select() -> poll(), nanosleep()

fix nested extern

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.

add/fix id tags

branches: 1.1.1;
Initial revision

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

fclose() open FILE * even though we are returning to fail anyway.
silences coverity, CID 1590 / run 5

KNF and WARNS=3

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

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

fix nested extern

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.

minor KNFify

WARNSify

add/fix id tags

branches: 1.1.1;
Initial revision

Create a new bsd.hostinit.mk file and put the build definitions for all host
programs there; make all Makefiles that use bsd.hostprog.mk include it.
Namely turn off MKREPRO and don't make lint, man pages, info files etc.
Remove the Makefile.inc files that contained these same settings, and
remove the settings from Makefile.host

branches: 1.12.94;
some makefile de-linting

MKfoo=no -> NOfoo=

Greatly simplify these by using <bsd.hostprog.mk>. Should do the same
for games/hack and games/monop and a few other places, but that requires
making subdirs for the tools. One day...

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

use HOST_CPPFLAGS not HOST_CFLAGS

use CPPFLAGS instead of CFLAGS

branches: 1.6.2;
Use proginstall:: to disable installing PROG.

Allow boggle to be cross-compiled; the mkdict and mkindex programs
are used only by the hosting system.

Yow, cvs commit in the wrong directory... Bad, Jason, no biscuit!

Allow this to be cross-compiled.

add/fix id tags

branches: 1.1.1;
Initial revision

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

KNF and WARNS=3

Fixed wrong use of the <ctype.h> functions by adding an explicit conversion
to unsigned char. Approved by christos.

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

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

Ensure trailing newline is present on error and warning messages (lost
in previous conversion from <err.h> functions to ISO C ones).

Don't use <sys/cdefs.h> __COPYRIGHT/__RCSID macros or <err.h> err/warnx
functions for host programs - use portable ANSI constructs instead.

Patches from Chris Demetriou.

Further warning fix for mkdict.

Warning cleanup for boggle's mkdict and mkindex.

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

minor KNFify

add/fix id tags

branches: 1.1.1;
Initial revision

Create a new bsd.hostinit.mk file and put the build definitions for all host
programs there; make all Makefiles that use bsd.hostprog.mk include it.
Namely turn off MKREPRO and don't make lint, man pages, info files etc.
Remove the Makefile.inc files that contained these same settings, and
remove the settings from Makefile.host

branches: 1.10.94;
some makefile de-linting

MKfoo=no -> NOfoo=

Greatly simplify these by using <bsd.hostprog.mk>. Should do the same
for games/hack and games/monop and a few other places, but that requires
making subdirs for the tools. One day...

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

use HOST_CPPFLAGS not HOST_CFLAGS

use CPPFLAGS instead of CFLAGS

branches: 1.4.2;
Use proginstall:: to disable installing PROG.

Allow boggle to be cross-compiled; the mkdict and mkindex programs
are used only by the hosting system.

add/fix id tags

branches: 1.1.1;
Initial revision

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

sprinkle static and prune some dead code

KNF and WARNS=3

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

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

Don't use <sys/cdefs.h> __COPYRIGHT/__RCSID macros or <err.h> err
function for host programs - use portable ANSI constructs instead.

Patches from Chris Demetriou.

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.

branches: 1.6.4;
Warning cleanup for boggle's mkdict and mkindex.

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

remove unused args from main(), as per PR#5867

minor KNFify

add/fix id tags

branches: 1.1.1;
Initial revision