From owner-freebsd-ports@FreeBSD.ORG Fri Jan 14 21:46:07 2011 Return-Path: Delivered-To: ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id BE92B1065679 for ; Fri, 14 Jan 2011 21:46:07 +0000 (UTC) (envelope-from rea@codelabs.ru) Received: from 0.mx.codelabs.ru (0.mx.codelabs.ru [144.206.177.45]) by mx1.freebsd.org (Postfix) with ESMTP id 4D1898FC1D for ; Fri, 14 Jan 2011 21:46:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=codelabs.ru; s=two; h=Sender:In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date; bh=Jz3yqdaXULGn6T/pKo3JxLvjwn0GQuhCltDXF1F7Eos=; b=iiZI5nRgeZfrYjbefyRJkCLZZUxqAl8+jL0d9kwPRbS3oLt3q2QqBkAeAM3Y1q4zvHy+lVsVj2S6hWIXDNHSyxcJE1cQy9MiYMFMDa28UJKNsCVuD7oTtsUetj3VkOow6EmtE73GmjTupYIyDJXgLHWqPymLkK5AU+zK15fx9LryD0944G7Q34qOJ/e9bhVesT4/2tGLIOQ98CnLY3quVYDxFo2yiObktdJsZnkfJQecBOzxeWY6D/kFYy4zT/5/Tan9D5O8Hy0tCPkk9BpBhusu7vA3d+38tt/S9XLTHT7mD2xh6TRg+qj70SSDyL0rQ6UQobnqaAoOs2WuftH3Ug==; Received: from amnesiac.at.no.dns (ppp91-78-117-186.pppoe.mtu-net.ru [91.78.117.186]) by 0.mx.codelabs.ru with esmtpsa (TLSv1:AES256-SHA:256) id 1PdrTB-0004W3-6V; Sat, 15 Jan 2011 00:46:05 +0300 Date: Sat, 15 Jan 2011 00:46:02 +0300 From: Eygene Ryabinkin To: joeb Message-ID: References: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="45Z9DzgjV8m4Oswq" Content-Disposition: inline In-Reply-To: Sender: rea@codelabs.ru Cc: ports@freebsd.org Subject: Re: Attention ports committers X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 Jan 2011 21:46:07 -0000 --45Z9DzgjV8m4Oswq Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Good day. Fri, Jan 14, 2011 at 01:18:52PM -0500, joeb wrote: > I would like to draw the attention of a ports committer to port=20 >=20 > http://www.freebsd.org/cgi/query-pr.cgi?pr=3Dports/148777 >=20 > If it needs further work please provide feedback. A couple of things about the port: - you can just use plain SF instead of a long to type ${MASTER_SITE_SOURCEFORGE}; moreover, plain SF/${PORTNAME} and no MASTER_SITE_SUBDIR will work too and save you one more line; - share/examples/qjail in the pkg-plist is better spelled as %%EXAMPLESDIR_REL%%; and its ${PREFIX}/share/examples/qjail in the Makefile should really be ${EXAMPLESDIR}; also you can consider using PORTEXAMPLES to use port's procedures to install the examples and to avoid installing them when NOPORTEXAMPLES is set; - it is better to remove the commented DISTFILES, since it is not needed thus just adds the visual clutter; - pkg-descr is a way too chatty (in my taste); it can be definitely stripped down to provide the idea about what qjail is and the whole text can be put to the project's Web site; - you can put your rc.d script into USE_RC_SUBR inside the Makefile; just put your scripts into WRKDIR during the build phase; this will enable your port to survive any modifications to the process of installation of the rc.d scripts, because they will be handled by the ports subsystem; - pkg-message can be stripped down as well: first line looks redundant, since people already know why they had installed the port; the other two lines, perhaps, can be condensed into "Read qjail-intro(8) for to get started." The said manual page has reference to the qjail(8), so people will know what to do after reading the introduction. And please, please, please, try to put your shar(1) archive somewhere to the Web and add the link to the PR: mail over gnats if not uuencoded properly makes attachments with partial quoted-printable stuff left, so download of, for example, http://www.freebsd.org/cgi/query-pr.cgi?prp=3D148777-3-shar&n=3D/shar-3.sh will give a non-working shar(1) archive for starters and when '=3D0A=3D' will be removed, one will need to dance with the QP '=3D' characters, newlines and alike. May I also add, your software's manual pages can be improved: - don't insert empty lines into the groff files, use the lone dot if you want the visual separator inside the groff sources; - use standard macros like .Pa, .Xr, .Nm, .Qq and others instead of doing bold font, quoting and other stuff by hand; - avoid unneeded usage of the bold attribute; for example, qjail-intro(8) needs no bold for the "4th generation", "user friendliness" and other stuff; - start each sentense inside the manual source from the new line: this will provide two spaces between sentences in the formatted manual pages and will ease the diffs between revisions of the makefile (for you and other developers of the qjail, because this way changes in the manual pages will show up in the diff output only inside their own sentenses; they won't have the parts of other sentences in the + and - lines and that's good); - you have many trailing spaces; whilst they typically do not harm, why to keep them? And for the qjail scripts themselves: - currently qjail is a single block with no subroutines; it _may_ be better to split it into logical parts, because it will be easier to read, navigate and understand for Other People (TM) ;)) - perhaps you'll be able to use exit codes from sysexits(3). - what does jail2 rc.d script? It terribly reminds me /etc/rc.d/jail and diffing the two one finds mostly cosmetic differences. So, what's the point of having jail2 at all? Sorry for a long letter and slightly mentorish tone: meant no offense, just wanted to improve the stuff. --=20 Eygene Ryabinkin ,,,^..^,,, [ Life's unfair - but root password helps! | codelabs.ru ] [ 82FE 06BC D497 C0DE 49EC 4FF0 16AF 9EAE 8152 ECFB | freebsd.org ] --45Z9DzgjV8m4Oswq Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (FreeBSD) iF4EAREIAAYFAk0wxBoACgkQFq+eroFS7Ps2sAD/VY23DVMnfOrR7g9jEirOrSUi gamwPmS9RKgjDlyk6RIA/RthfANDHFz5nTLL6x4eZnD8aHKLeJnyP7RGrUsSvELL =/te9 -----END PGP SIGNATURE----- --45Z9DzgjV8m4Oswq--