install(1): Fix 2.5 y/o verbose bug, use static functions
authorJohn Marino <draco@marino.st>
Thu, 6 Feb 2014 13:33:01 +0000 (14:33 +0100)
committerJohn Marino <draco@marino.st>
Thu, 6 Feb 2014 13:36:23 +0000 (14:36 +0100)
commita88cc817133093499245c4dd191df21b50624995
treecfd9e386dbdc7bff5d5d4eea32b82199b2881022
parent3e788e0600e5f38d915223ae0eb11aa95d7bfcbc
install(1): Fix 2.5 y/o verbose bug, use static functions

A bug was introduced by this commit to add the -D option:

a2bc4e56d4ab068fea12d65d13740b7a600769a8 (7 SEP 11)

It changed the -v verbose option from standalone to expecting input and
as a result, any use of -v caused the usage screen to come up.  Fixed.
Additionally, the pathnames.h header is unused, removed now.
Finally, several functions were turned into static functions and explicit
void casts were used when functions normally return integers.  The last
two change sets converge with FreeBSD.

xinstall has not been seriously updated in a long time and has fallen
behind BSD counterparts.  Syncing with FreeBSD may take several steps as
there is significant divergence and options (e.g. -l) do different things
on FreeBSD
usr.bin/xinstall/pathnames.h [deleted file]
usr.bin/xinstall/xinstall.c