Sun, 26 Jun 2016 22:31:55 +0200 Adam Wolk <[email protected]>
> > Same diff but with an additional change to
> > ports/infrastructure/db/user.list and using the same ID.
An underscore informs me it's a program Who brought the user, correct?
Is it any reasonable consideration to make the user names any package,
(here a port) brings up on the system match the _program (_port) name?
Is it also worth making packages (ports) users/groups start with a _p
so they end up together when listed alphabetically, e.g. _pdescent2 ?
NB: I am only considering the system maintenance factors, and don't
insist on anything changed, but please let me know what is the norm.
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/games/yquake2/Makefile,v
> retrieving revision 1.3
> diff -u -p -r1.3 Makefile
> --- Makefile 16 Mar 2016 16:46:32 -0000 1.3
> +++ Makefile 26 Jun 2016 20:23:55 -0000
> @@ -4,12 +4,13 @@ ONLY_FOR_ARCHS= i386 amd64 sparc64
>
> COMMENT= Yamagi Quake II
> N= yquake2
> -V= 5.32
> +V= 5.34
> PKGNAME= ${N}-${V}
> DISTNAME= quake2-${V}
> CATEGORIES= games
>
> HOMEPAGE= http://www.yamagi.org/quake2/
> +MAINTAINER= Adam Wolk <[email protected]>
> MASTER_SITES= http://deponie.yamagi.org/quake2/
> EXTRACT_SUFX= .tar.xz
>
> @@ -25,11 +26,12 @@ LIB_DEPENDS= audio/libvorbis \
> MAKE_ENV+= VERBOSE=1
> USE_GMAKE= Yes
>
> +MAKE_FLAGS = config WITH_SYSTEMWIDE=yes WITH_SYSTEMDIR=${PREFIX}/share/${N}
> +
> do-install:
> - ${INSTALL_SCRIPT} ${FILESDIR}/yquake2 ${PREFIX}/bin/
> ${INSTALL_DATA_DIR} ${PREFIX}/share/${N}
> ${INSTALL_PROGRAM} ${WRKBUILD}/release/{quake2,q2ded} \
> - ${PREFIX}/share/${N}/
> + ${PREFIX}/bin/
> ${INSTALL_DATA_DIR} ${PREFIX}/share/${N}/baseq2
> ${INSTALL_PROGRAM} ${WRKBUILD}/release/baseq2/game.so \
> ${PREFIX}/share/${N}/baseq2/
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/games/yquake2/distinfo,v
> retrieving revision 1.1.1.1
> diff -u -p -r1.1.1.1 distinfo
> --- distinfo 17 Jan 2016 15:18:31 -0000 1.1.1.1
> +++ distinfo 26 Jun 2016 20:23:55 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (quake2-5.32.tar.xz) = v8eAMlSp0iiIVU1a8lL//iEts9qwYxY3u5BFhhuOUIw=
> -SIZE (quake2-5.32.tar.xz) = 1692720
> +SHA256 (quake2-5.34.tar.xz) = gOEZPGM9vuh7n8uGQwafm9pEOqZVUcjzUYcBNsM9ONQ=
> +SIZE (quake2-5.34.tar.xz) = 1702984
> Index: files/yquake2
> ===================================================================
> RCS file: files/yquake2
> diff -N files/yquake2
> --- files/yquake2 17 Jan 2016 15:18:31 -0000 1.1.1.1
> +++ /dev/null 1 Jan 1970 00:00:00 -0000
> @@ -1,3 +0,0 @@
> -#!/bin/sh
> -cd /usr/local/share/yquake2
> -exec /usr/local/share/yquake2/quake2 "$@"
> Index: pkg/PLIST
> ===================================================================
> RCS file: /cvs/ports/games/yquake2/pkg/PLIST,v
> retrieving revision 1.1.1.1
> diff -u -p -r1.1.1.1 PLIST
> --- pkg/PLIST 17 Jan 2016 15:18:31 -0000 1.1.1.1
> +++ pkg/PLIST 26 Jun 2016 20:23:55 -0000
> @@ -1,8 +1,17 @@
> @comment $OpenBSD: PLIST,v 1.1.1.1 2016/01/17 15:18:31 bmercer Exp $
> -bin/yquake2
> +@newgroup _q2:779
> +@newuser _q2:779:_q2:daemon:Yamagi Quake II Server:/var/q2:/sbin/nologin
> +@bin bin/q2ded
> +@bin bin/quake2
> share/doc/pkg-readmes/${FULLPKGNAME}
> share/yquake2/
> share/yquake2/baseq2/
> share/yquake2/baseq2/game.so
> -@bin share/yquake2/q2ded
> -@bin share/yquake2/quake2
> +@mode 750
> +@owner _q2
> +@group _q2
> +@sample /var/q2/
> +@owner
> +@group
> +@mode
> +@rcscript ${RCDIR}/q2ded
> Index: pkg/q2ded.rc
> ===================================================================
> RCS file: pkg/q2ded.rc
> diff -N pkg/q2ded.rc
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ pkg/q2ded.rc 26 Jun 2016 20:23:55 -0000
> @@ -0,0 +1,15 @@
> +#!/bin/sh
> +#
> +# $OpenBSD: $
> +
> +daemon="${TRUEPREFIX}/bin/q2ded"
> +daemon_user="_q2"
> +
> +. /etc/rc.d/rc.subr
> +
> +pexp="${daemon}.*"
> +
> +rc_bg=YES
> +rc_reload=NO
> +
> +rc_cmd $1
> Index: user.list
> ===================================================================
> RCS file: /cvs/ports/infrastructure/db/user.list,v
> retrieving revision 1.275
> diff -u -p -r1.275 user.list
> --- user.list 2 Jun 2016 18:32:26 -0000 1.275
> +++ user.list 26 Jun 2016 20:24:13 -0000
> @@ -287,3 +287,4 @@ id user group port options
> 776 _ioq3 _ioq3 games/ioquake3
> 777 _svnserve _svnserve devel/subversion
> 778 _gitdaemon _gitdaemon devel/git
> +779 _q2 _q2 games/yquake2
>
>