games/xbill: Updated for version 2.1

5 files changed, 30 insertions, 15 deletions

diff --git a/games/xbill/README b/games/xbill/README
index c47fc79c2d..2012e4db6b 100644
--- a/games/xbill/README
+++ b/games/xbill/README
@@ -10,8 +10,11 @@ which obviously allows cheating ones hi-score :-(
 Probably this came about since GTK+ doesn't allow setgid?
  
 In the install-script we move xbill to xbill-bin and install
-a wrapper thingy as xbill instead, with setuid and video group
-executable filesystem perms. There we change groups to games.
-
-In the resulting package /var/xbill is group-writable only...
+a wrapper thingy as xbill instead. In the resulting package
+/var/games/xbill is group-writable for 'games' only...
 
+Note on capabilities: FCAPS=true
+This is the default now. It maybe disabled by feeding a 'false'
+value to the build script and the wrapper will be installed with
+setuid and video group executable filesystem perms. As before.
+Just comment it out, if you don't care for any of this stuff :-)
diff --git a/games/xbill/doinst.sh b/games/xbill/doinst.sh
index ea4ae6fbe9..53cf51a704 100644
--- a/games/xbill/doinst.sh
+++ b/games/xbill/doinst.sh
@@ -18,7 +18,3 @@ config var/games/xbill/scores.new
 chgrp -R games var/games/xbill
 chmod -R g+w,o-w var/games/xbill
 
-# Change privs on our little wrapper
-chgrp video usr/bin/xbill
-chmod 4750 usr/bin/xbill
-
diff --git a/games/xbill/xbill-wrapper.c b/games/xbill/xbill-wrapper.c
index a98cc37aee..119e58c014 100644
--- a/games/xbill/xbill-wrapper.c
+++ b/games/xbill/xbill-wrapper.c
@@ -1,4 +1,4 @@
-/* Written by Menno E. Duursma for use with xbill */
+/* Written by Menno Duursma for use with xbill */
 
 /*
  * This program is free software. It comes without any warranty.
@@ -62,7 +62,7 @@ int main(int argc, char *argv[], char *envp[])
 			getuid(),
 			strerror(errno));
 		exit(EXIT_FAILURE);
-		}
+	}
 
         /* tell the viewers wat is going to happen */
         fprintf(stderr, "Starting %s with uid = %d, gid = %d\n",
diff --git a/games/xbill/xbill.SlackBuild b/games/xbill/xbill.SlackBuild
index bfab3c2913..3994de1e14 100644
--- a/games/xbill/xbill.SlackBuild
+++ b/games/xbill/xbill.SlackBuild
@@ -1,7 +1,8 @@
 #!/bin/sh
 
 # Slackware build script for XBill
-# Written by Menno E. Duursma <druiloor@zonnet.nl>
+
+# Written by Menno Duursma <druiloor@zonnet.nl>
 
 # This program is free software. It comes without any warranty.
 # Granted WTFPL, Version 2, as published by Sam Hocevar. See
@@ -12,7 +13,7 @@
 PRGNAM=xbill
 VERSION=${VERSION:-2.1}
 ARCH=${ARCH:-i486}
-BUILD=${BUILD:-1}
+BUILD=${BUILD:-2}
 TAG=${TAG:-_SBo}
 
 CWD=$(pwd)
@@ -20,13 +21,19 @@ TMP=${TMP:-/tmp/SBo}
 PKG=$TMP/package-$PRGNAM
 OUTPUT=${OUTPUT:-/tmp}
 
+# On capability enabled filesystems this may be enabled,
+# i'd guess if this box runs x-apps, it probably has...
+FCAPS=${FCAPS:-true}
+
 if [ "$ARCH" = "i486" ]; then
   SLKCFLAGS="-O2 -march=i486 -mtune=i686"
 elif [ "$ARCH" = "i686" ]; then
   SLKCFLAGS="-O2 -march=i686 -mtune=i686"
+elif [ "$ARCH" = "x86_64" ]; then
+  SLKCFLAGS="-O2 -fPIC"
 fi
 
-set -e
+set -e # Exit on most errors
 
 rm -rf $PKG
 mkdir -p $TMP $PKG $OUTPUT
@@ -59,7 +66,7 @@ make install DESTDIR=$PKG
 mv $PKG/var/games/xbill/scores $PKG/var/games/xbill/scores.new
 
 # Do a little trick to disallow cheating, hopefully
-# Note: we do the privs stuff in doinst.sh for target system
+# Note: we do the privs stuff below at doinst.sh
 cp -a $PKG/usr/bin/xbill $PKG/usr/bin/xbill-bin
 gcc -Wall $SLKCFLAGS $CWD/xbill-wrapper.c -o $PKG/usr/bin/xbill
 
@@ -87,5 +94,14 @@ mkdir -p $PKG/install
 cat $CWD/slack-desc > $PKG/install/slack-desc
 cat $CWD/doinst.sh > $PKG/install/doinst.sh
 
+if [ "$FCAPS" != "false" ]; then
+  chmod 0751 $PKG/usr/bin/xbill
+  echo 'setcap "cap_setgid=ep" usr/bin/xbill' >> $PKG/install/doinst.sh
+else
+  # Install setuid-root
+  chgrp video usr/bin/xbill
+  chmod 4750 $PKG/usr/bin/xbill
+fi
+
 cd $PKG
 /sbin/makepkg -l y -c n $OUTPUT/$PRGNAM-$VERSION-$ARCH-$BUILD$TAG.tgz
diff --git a/games/xbill/xbill.info b/games/xbill/xbill.info
index 2709681798..773bf57ef8 100644
--- a/games/xbill/xbill.info
+++ b/games/xbill/xbill.info
@@ -5,4 +5,4 @@ DOWNLOAD="http://www.xbill.org/download/xbill-2.1.tar.gz"
 MD5SUM="585e4866b15255a24203db9959407b2f"
 MAINTAINER="Menno E. Duursma"
 EMAIL="druiloor@zonnet.nl"
-APPROVED="David Somero"
\ No newline at end of file
+APPROVED="dsomero"