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

Reply via email to