From owner-cvs-ports@FreeBSD.ORG Thu Oct 27 06:02:26 2011 Return-Path: Delivered-To: cvs-ports@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 944F8106564A; Thu, 27 Oct 2011 06:02:26 +0000 (UTC) (envelope-from sunpoet@sunpoet.net) Received: from sunpoet.net (sunpoet.net [220.133.12.240]) by mx1.freebsd.org (Postfix) with ESMTP id 1F0228FC19; Thu, 27 Oct 2011 06:02:26 +0000 (UTC) Received: by sunpoet.net (Postfix, from userid 1000) id 7021C5770; Thu, 27 Oct 2011 14:01:55 +0800 (CST) Date: Thu, 27 Oct 2011 14:01:55 +0800 From: Sunpoet Po-Chuan Hsieh To: Doug Barton Message-ID: <20111027060155.GB38115@bonjour.sunpoet.net> References: <201110261620.p9QGK0vV010389@repoman.freebsd.org> <4EA86333.8090804@FreeBSD.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="cvVnyQ+4j833TQvp" Content-Disposition: inline In-Reply-To: <4EA86333.8090804@FreeBSD.org> Organization: The FreeBSD Project User-Agent: Mutt/1.5.21 (2010-09-15) Cc: Olivier Cochard-Labbe , cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/www/shellinabox Makefile ports/www/shellinabox/files shellinaboxd.in X-BeenThere: cvs-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 27 Oct 2011 06:02:26 -0000 --cvVnyQ+4j833TQvp Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Oct 26, 2011 at 12:44:51PM -0700, Doug Barton wrote: > On 10/26/2011 09:20, Sunpoet Po-Chuan Hsieh wrote: > > sunpoet 2011-10-26 16:20:00 UTC > >=20 > > FreeBSD ports repository > >=20 > > Modified files: > > www/shellinabox Makefile=20 > > www/shellinabox/files shellinaboxd.in=20 > > Log: > > - Unset shellinaboxd_user and shellinaboxd_group at the end of rc scr= ipt to > > avoid unnecessary su(1) initiated by rc.subr >=20 > I solved this in the named rc.d script by using _uid, which is not > special to rc.subr; instead of _user, which is. Also, the following > should probably be changed: >=20 > if [ "${shellinaboxd_certfile}" =3D "" ]; then >=20 > to >=20 > if [ -n "${shellinaboxd_certfile}" ]; then >=20 > Otherwise this script looks real good, thanks. :) Hi Doug, Using _uid/_gid is also a feasible solution to avoid unnecessary su. However, it would require users to change their rc.conf. That's why I choose to keep the old variable names. :) For shellinaboxd_certfile part, it's definitely better to use -z. Please commit your patch if maintainer approves. Thanks. Regards, sunpoet >=20 > > - Pet portlint > > - Bump PORTREVISION for package change > > =20 > > Reported by: ed (in the followup of ports/151314) > > Approved by: Olivier Cochard-Labbe (maintaine= r) > > =20 > > Revision Changes Path > > 1.6 +4 -4 ports/www/shellinabox/Makefile > > 1.6 +4 -1 ports/www/shellinabox/files/shellinaboxd.in > >=20 > > http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/www/shellinabox/Makefile.di= ff?&r1=3D1.5&r2=3D1.6&f=3Dh > > http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/www/shellinabox/files/shell= inaboxd.in.diff?&r1=3D1.5&r2=3D1.6&f=3Dh > >=20 >=20 > --=20 >=20 > Nothin' ever doesn't change, but nothin' changes much. > -- OK Go >=20 > Breadth of IT experience, and depth of knowledge in the DNS. > Yours for the right price. :) http://SupersetSolutions.com/ >=20 --=20 Sunpoet Po-Chuan Hsieh 4096R/CC57E36B 8AD8 68F2 7D2B 0A10 7E9B 8CC0 DC44 247E CC57 E36B http://people.FreeBSD.org/~sunpoet/pgpkeys.txt --cvVnyQ+4j833TQvp Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (FreeBSD) iQIcBAEBAgAGBQJOqPPSAAoJENxEJH7MV+Nr53MP/2Z+87LqFrmwKyY+X/sZ2D7/ YSsLCU1UodHG5y9bB/lTBntVX72CGpZFGqxqX3CpBHRdyJJHiH8wmqobllbd2w10 LpfHvXgvevSiWhor0P4H5SPRILe7zALX2Uh9vT2NdWQuAkp4z/FM0SIIvVv6bImD y5DX7KtQoibU8E4RxwBLOh8lEWRyNBTWAGpQHlHaUEIMcmyt8XBlYjO54qGiXYGc U+lqgIueyy9IVfF6TYyPCv8dc0qW7BtSgp4jimOAt2wARF/e2lx2PVqKkGGGM/Lf v3GoAyvBPD+N6D8gxVoaq4yKwGPdqjTP0Sv+Rtfp/EYaY8Nn7YcfAzZzzrYpN16l gt0BlGM5MEthpIPSIkjWtDbQXwMkSEvCXPNMf7sXqMHxR9QQGEOIW+vLbdUQIpo4 Ndh9KWCtpMvfD7Uj4d2A92rx4VbqtRuAeH3KT3kQrsMXhZo6q4HJuc4S6sFf8k6D KTqnPk7lh3Dgr8spNk7qPsP9jqXn2t0cLjkq8dKMo6cqqOVCOgnMFa0dPhlYXZvK in4iLVL9F0DzZib19YNpb/tOQtZ4IyVgoSCR/B9PIIIgA5Vhd5btmq0oTnK2aaDY 9pqGDiQ1jMtmMSLHZfy6FyY6ydX7p/Un9NXtlLimv3FZ/A7K9B2A5o8PY7r7LRPz bW6fNeRukBjqr4YsRbzx =j/6a -----END PGP SIGNATURE----- --cvVnyQ+4j833TQvp--