From owner-svn-ports-all@freebsd.org Tue Jan 23 14:47:48 2018 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 3D5D9ED7457; Tue, 23 Jan 2018 14:47:48 +0000 (UTC) (envelope-from mat@aragorn.in.absolight.net) Received: from prod2.absolight.net (prod2.absolight.net [79.143.243.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "plouf.absolight.net", Issuer "CAcert Class 3 Root" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id BAF547F0FB; Tue, 23 Jan 2018 14:47:46 +0000 (UTC) (envelope-from mat@aragorn.in.absolight.net) Received: from prod2.absolight.net (localhost [127.0.0.1]) by prod2.absolight.net (Postfix) with ESMTP id 06DA4BDD37; Tue, 23 Jan 2018 15:47:45 +0100 (CET) Received: from gw.in.absolight.net (gw-ecl.in.absolight.net [79.143.241.69]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "gw.in.absolight.net", Issuer "CA Cert Signing Authority" (not verified)) by prod2.absolight.net (Postfix) with ESMTPSA id DF672BDD2D; Tue, 23 Jan 2018 15:47:44 +0100 (CET) Received: from aragorn.in.absolight.net (aragorn.in.absolight.net [79.143.241.225]) by gw.in.absolight.net (Postfix) with ESMTP id 15382612C; Tue, 23 Jan 2018 15:47:44 +0100 (CET) Received: by aragorn.in.absolight.net (Postfix, from userid 1000) id E085A3CCA3; Tue, 23 Jan 2018 15:47:43 +0100 (CET) Date: Tue, 23 Jan 2018 15:47:43 +0100 From: Mathieu Arnold To: Alex Dupre Cc: Alexey Dokuchaev , svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org Subject: Re: svn commit: r459751 - in head/lang: . solidity solidity/files Message-ID: <20180123144743.scc42sitwjhvxspv@aragorn.in.absolight.net> References: <201801231358.w0NDwMa8074241@repo.freebsd.org> <20180123140938.GA73049@FreeBSD.org> <57ffd7bd-6a3f-b315-b582-17d9dccee5a8@FreeBSD.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="755f4kp4bfgi6t6n" Content-Disposition: inline In-Reply-To: <57ffd7bd-6a3f-b315-b582-17d9dccee5a8@FreeBSD.org> User-Agent: NeoMutt/20171215 X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 23 Jan 2018 14:47:48 -0000 --755f4kp4bfgi6t6n Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jan 23, 2018 at 03:31:03PM +0100, Alex Dupre wrote: > Alexey Dokuchaev wrote: > > On Tue, Jan 23, 2018 at 01:58:22PM +0000, Alex Dupre wrote: > >> New Revision: 459751 > >> URL: https://svnweb.freebsd.org/changeset/ports/459751 > >> > >> Log: > >> The Solidity Contract-Oriented Programming Language > >> =20 > >> WWW: https://github.com/ethereum/solidity > >> > >> Added: head/lang/solidity/Makefile > >> ... >=20 > Hi Alexey, I often agree with your comments, but this time I have to > strongly disagree. >=20 > % portlint -tac solidity/ > looks fine. >=20 > >> +COMMENT=3D Solidity Contract-Oriented Programming Language > >=20 > > COMMENT=3D Contract-oriented programming language >=20 > Come on, then the comment for the scala port should be "Programming > Language"? >=20 > >=20 > >> +OPTIONS_DEFINE=3D SMT > >> +SMT_DESC=3D SMT Checker > >> +SMT_LIB_DEPENDS=3Dlibz3.so:math/z3 > >> + > >> +USES=3D cmake > >=20 > > Misplaced USES. >=20 > I don't see what's misplaced here, portlint doesn't seem to agree with yo= u. portlint is a third party static lint tool, it does not understand what you does, it just tells you when it is very very bad (when it does not get it wrong.) https://www.freebsd.org/doc/en/books/porters-handbook/porting-order.html > >> +CMAKE_ARGS+=3D -DUSE_LD_GOLD=3DOFF -DTESTS=3DOFF \ > >> + -DJSONCPP_INCLUDE_DIR:STRING=3D"${LOCALBASE}/include/jsoncpp" \ > >> + -DJSONCPP_LIB_DIR:STRING=3D"${LOCALBASE}/lib" > >=20 > > And CMAKE_ARGS (also bogus +=3D). >=20 > Really? >=20 > >> +pre-patch: > >> + @${CP} ${FILESDIR}/jsoncpp.cmake ${WRKSRC}/cmake/ > >> + @${REINPLACE_CMD} '/EthCcache/d' ${WRKSRC}/CMakeLists.txt > >=20 > > Excessive indentation of target recipe (commands). >=20 > Seriously? >=20 > >> Added: head/lang/solidity/pkg-descr > >> ... > >> @@ -0,0 +1,3 @@ > >> +The Solidity Contract-Oriented Programming Language > >> + > >> +WWW: https://github.com/ethereum/solidity > >=20 > > Bad port description (verbatim copy of COMMENT), tells nothing about the > > port (violation of PHB Section 3.2.1). > This is the only thing I agree on, changed. >=20 > --=20 > Alex Dupre >=20 >=20 --755f4kp4bfgi6t6n Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQKTBAABCgB9FiEEOraXidLtEhBkQLpbOkUW81GDzkgFAlpnSw5fFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDNB QjY5Nzg5RDJFRDEyMTA2NDQwQkE1QjNBNDUxNkYzNTE4M0NFNDgACgkQOkUW81GD zkiGJg//YaUWtWHK9BVS6fkFIkEwO0yGRmfkXiVzHoLASP9J3ClNqh820RUgKK6J 10YkWNlR9EmlfUZozqewIf4A0hCI2b0qU7IkNoQQyQ7to9k31PjB3pU11luXoPKz aW8KScHBBhalqXbJTZ7ldSWNDqxvS67JQh+Ybqz2uUKXoeozVxm5pU3qUac35xN+ uhd6OkTWLtU60CIq71rmXrWAGz7SkArSxMCdQaN1tTb6qrBljvPS1VGNVk6RMbbS 2zUCOQ5dLULVXBQ0K+mJVESOGjB0WeygKUQEOC5CWd0Z2Gr91/tk1WWSpKeBJipv JS99l9VDOKIkwA0ujk2EpzhamyK+W6IcecBqtUehiLPWk7B6/OLjziK8a7ihawMA K7SQAyIFqK4MijAWW4a1c6psQF0FQ44ZI/1piyPX6kmF+M0DC/iXtyd4jnhUaJRX 1bjBFpH1pJl/qGi81jdJ9FqI/ORtvPL3F5HeCWmmOMSxTTzrW/66L840l4P7txmp X/ooB2HDVJ4we0OfOAcSAyyB5y0botl8R7QgkcsZXzu5pKfP8dKYEV+uEJQ0OgKO 3o0Td+1a6wJ3B1SSQhxSJW7SY3yVaACT7SWzYgjzZi/Ky5288HC1vbqVCvjcDZ1n MsYRVelC+OjEskV+k7vmwgF4l9MENUNg5xawostP1L4aR0XvFUY= =htnA -----END PGP SIGNATURE----- --755f4kp4bfgi6t6n--