From what I can tell, install.sh supported all of the install(1)

options since revision 1.2, so removed the XXX comment now.

Fixed bogus test(1) expression that would be evaluated as a null
string test if script is run without arguments, compacted option
lists.

1 files changed, 3 insertions, 4 deletions

diff --git a/tools/install.sh b/tools/install.sh
index c578dda..899d7e2 100644
--- a/tools/install.sh
+++ b/tools/install.sh
@@ -29,11 +29,10 @@
 # $FreeBSD$
 
 # parse install's options and ignore them completely.
-# XXX - not all options are recognised.
-while [ ! -z $1 ]; do
+while [ $# -gt 0 ]; do
     case $1 in
-    -b | -C | -c | -M | -p | -S | -s) shift;;
-    -B | -f | -g | -m | -o) shift; shift;;
+    -[bCcMpSs]) shift;;
+    -[Bfgmo]) shift; shift;;
     *) break;
     esac
 done