From owner-svn-ports-head@FreeBSD.ORG Fri May 8 18:17:41 2015 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id EC88531D; Fri, 8 May 2015 18:17:41 +0000 (UTC) Received: from smtp.infracaninophile.co.uk (smtp6.infracaninophile.co.uk [IPv6:2001:8b0:151:1:3cd3:cd67:fafa:3d78]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "smtp.infracaninophile.co.uk", Issuer "ca.infracaninophile.co.uk" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 86B371DE4; Fri, 8 May 2015 18:17:41 +0000 (UTC) Received: from zero-gravitas.local (vpn-1.adestra.com [46.236.37.122]) (authenticated bits=0) by smtp.infracaninophile.co.uk (8.15.1/8.15.1) with ESMTPSA id t48IDctA024784 (version=TLSv1.2 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Fri, 8 May 2015 19:13:39 +0100 (BST) (envelope-from matthew@FreeBSD.org) Authentication-Results: smtp.infracaninophile.co.uk; dmarc=none header.from=FreeBSD.org DKIM-Filter: OpenDKIM Filter v2.9.2 smtp.infracaninophile.co.uk t48IDctA024784 Authentication-Results: smtp.infracaninophile.co.uk/t48IDctA024784; dkim=none reason="no signature"; dkim-adsp=none; dkim-atps=neutral X-Authentication-Warning: lucid-nonsense.infracaninophile.co.uk: Host vpn-1.adestra.com [46.236.37.122] claimed to be zero-gravitas.local Message-ID: <554CFCCB.8070800@FreeBSD.org> Date: Fri, 08 May 2015 19:13:31 +0100 From: Matthew Seaman User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Bryan Drewery , Sunpoet Po-Chuan Hsieh CC: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r385776 - in head/sysutils: . rubygem-hiera-file References: <201505081305.t48D5ULi055835@svn.freebsd.org> <554CD6C9.5000708@FreeBSD.org> <554CDBE8.7020703@FreeBSD.org> In-Reply-To: <554CDBE8.7020703@FreeBSD.org> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="IP3E75A82wLd6HfBDNSlgRsIPLvciTJf0" X-Virus-Scanned: clamav-milter 0.98.6 at lucid-nonsense.infracaninophile.co.uk X-Virus-Status: Clean X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 08 May 2015 18:17:42 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --IP3E75A82wLd6HfBDNSlgRsIPLvciTJf0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 2015/05/08 16:53, Bryan Drewery wrote: > On 5/8/2015 10:43 AM, Sunpoet Po-Chuan Hsieh wrote: >> On Fri, May 8, 2015 at 11:31 PM, Bryan Drewery > > wrote: >> >> On 5/8/2015 9:45 AM, Sunpoet Po-Chuan Hsieh wrote: >> > On Fri, May 8, 2015 at 9:05 PM, Matthew Seaman >> > >> wrote= : >> > >> > Author: matthew >> > Date: Fri May 8 13:05:29 2015 >> > New Revision: 385776 >> > URL: https://svnweb.freebsd.org/changeset/ports/385776 >> > >> > Log: >> > A data backend for Hiera that can return the content of >> whole files. >> > >> > WWW: https://github.com/adrienthebo/hiera-file >> > >> > Added: >> > head/sysutils/rubygem-hiera-file/ >> > head/sysutils/rubygem-hiera-file/Makefile (contents, pro= ps >> changed) >> > head/sysutils/rubygem-hiera-file/distinfo (contents, pro= ps >> changed) >> > head/sysutils/rubygem-hiera-file/pkg-descr (contents, >> props changed) >> > Modified: >> > head/sysutils/Makefile >> > >> > Modified: head/sysutils/Makefile >> > =20 >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D >> > --- head/sysutils/Makefile Fri May 8 13:02:39 2015 >> > (r385775) >> > +++ head/sysutils/Makefile Fri May 8 13:05:29 2015 >> > (r385776) >> > @@ -836,6 +836,7 @@ >> > SUBDIR +=3D rubygem-hammer_cli_foreman_salt >> > SUBDIR +=3D rubygem-hammer_cli_foreman_ssh >> > SUBDIR +=3D rubygem-hiera >> > + SUBDIR +=3D rubygem-hiera-file >> > SUBDIR +=3D rubygem-librarian-puppet >> > SUBDIR +=3D rubygem-log4r >> > SUBDIR +=3D rubygem-logify >> > >> > Added: head/sysutils/rubygem-hiera-file/Makefile >> > =20 >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D >> > --- /dev/null 00:00:00 1970 (empty, because file is newl= y >> added) >> > +++ head/sysutils/rubygem-hiera-file/Makefile Fri May 8 >> 13:05:29 >> > 2015 (r385776) >> > @@ -0,0 +1,18 @@ >> > +# Created by: Matthew Seaman >> > +# $FreeBSD$ >> > + >> > +PORTNAME=3D hiera-file >> > +PORTVERSION=3D 1.1.1 >> > +CATEGORIES=3D sysutils rubygems >> > +MASTER_SITES=3D RG >> > + >> > +MAINTAINER=3D matthew@FreeBSD.org >> > +COMMENT=3D Data backend for Hiera that returns whole = files >> > + >> > +USE_RUBY=3D yes >> > +USE_RUBYGEMS=3D yes >> > +RUBYGEM_AUTOPLIST=3D yes >> > + >> > +RUN_DEPENDS=3D >> > ${PKGNAMEPREFIX}hiera>=3D0:${PORTSDIR}/sysutils/rubygem-hie= ra >> > >> > >> > RUN_DEPENDS should appear earlier. :) >> > >> >> Why? >> >> >> % portlint -abct >> WARN: Makefile: Consider defining LICENSE. >> WARN: Makefile: "RUN_DEPENDS" has to appear earlier. >> 0 fatal errors and 2 warnings found. >=20 > This rule serves no purpose. The port was pretty much cut'n'paste from sysutils/rubygem-hiera. Besides, where in the Makefile would be considered suitable by portlint? Above all the USE_FOO=3D stuff? That's going to generate even worse complaints. Oh, I get it. It thinks the RUBYGEM_AUTOPLIST line should occur later. Except that the rational place to put that is right next to the 'USE_RUBYGEMS' line. portlint isn't guarranteed to be perfect, and in this case, I think it is complaining unnecessarily. Cheers, Matthew --IP3E75A82wLd6HfBDNSlgRsIPLvciTJf0 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.22 (Darwin) Comment: GPGTools - https://gpgtools.org iQJ8BAEBCgBmBQJVTPzMXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQxOUYxNTRFQ0JGMTEyRTUwNTQ0RTNGMzAw MDUxM0YxMEUwQTlFNEU3AAoJEABRPxDgqeTnxPkP/0zXnM02ESWzSZITyQaTX1Q2 AmP0womy7kbP6fyrx3uKJx2D2ugh1vLR13j6NPz33+YdBesTvhtpxK8f0uNsTvyx PutB0eO1pPu0DcKgjg5Pc2+H2gDU5yQh4HtfvWsEsHsk52RdKrJ31aw47Eap14n+ LXU4gvwLfx02zAf6Sz18khj6cecMfhepOwvMNoX67r6ANV+iU/OUhhSQSieS6dD8 331srWO6eAL24a8pNvYju7gBVGGrC/O5UYcoqaqvOfd1FkEWcN6OdNglZQ1wWBVC OPSEMBa7k7V2STnWWrkAYv03F1vzEAwZzeyERkXSO9u6H2v3LbXXu/JG1URyt41z pQ0XrNnB37bX9wcYHRamwW4ew09Br6HcYYAhZc+3S25A+ul03wANIT5QDLBf+K26 GHm6Tk+fPhVj8ZBUmhVn1U+R1UXOe48sg7A5eSYPLi4wguMW+b/sZ/t/v+HDlRqf B38Nop4J1boL9kWd2oItFL2erEfXIAjI2Pah2ozhrYy7BAoyZdNVuq8JAPkRN1uZ lLp15/Iv2tF+NlHtINIhue2DVztof+fX355p7DDXWPVgJ2m9fGNKAJntU+Af3tvf VbHrVMbkzPNl3wRFGj4qsQPdUqpDw2hLW3YGr/8v1s8QyjFuRuU680rUdmqSb3w4 0ftqbYTiQ/HT07QgGgOl =TqF4 -----END PGP SIGNATURE----- --IP3E75A82wLd6HfBDNSlgRsIPLvciTJf0--