From owner-svn-ports-all@freebsd.org Sun Oct 13 06:27:33 2019 Return-Path: Delivered-To: svn-ports-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 3FA16E65A2; Sun, 13 Oct 2019 06:27:33 +0000 (UTC) (envelope-from tobik@freebsd.org) Received: from wout4-smtp.messagingengine.com (wout4-smtp.messagingengine.com [64.147.123.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 46rWvT0lHWz4Zj9; Sun, 13 Oct 2019 06:27:32 +0000 (UTC) (envelope-from tobik@freebsd.org) Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.west.internal (Postfix) with ESMTP id B6FCD4E4; Sun, 13 Oct 2019 02:27:31 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute7.internal (MEProxy); Sun, 13 Oct 2019 02:27:32 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=EJ6CX7 j5TngOHJ6aOxpBnZrywvCoOZQM8vUxLgo5rHo=; b=hOwedRI8h5/3gLjQ1WBV4r psP/zkFmgbCR6nEEXQJcT/zxQiWH4xCin87emqfCqmu8QzFUBHnm8toKlqlX1QeU 7Uf+f4zFEbWKNWdljNWjJ5rT7z/17qgFGs2s0FEGy77h9r09ytJTC+I1DfteKLqp 5J5WS49rQyBNzDW/1Xaq5Wb5geFFQJFW08WOmcJHVndK1OIbx5eNZcidLSKeORyy G5oe76uPea4+00jEYU7mE+nc1BNQMbJ6hyiKoeufs8y95bh4Cz7sGlrp/lHm7eEe 8gNKu13H7nDG4i089TuJ5PFLrdTDAQ1IILuw0+B0cB8iCW1vli/pMHjHlKEQUHdw == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedufedrieekgddutdeiucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjfgesghdtreertdervdenucfhrhhomhepvfhosghi rghsucfmohhrthhkrghmphcuoehtohgsihhksehfrhgvvggsshgurdhorhhgqeenucffoh hmrghinhepfhhrvggvsghsugdrohhrghdpphhorhhtrdhmkhdpghhithhhuhgsrdgtohhm pdgtohhnfhdrihhtnecukfhppeelhedrledtrddvvdeirdduudenucfrrghrrghmpehmrg hilhhfrhhomhepthhosghikhesfhhrvggvsghsugdrohhrghenucevlhhushhtvghrufhi iigvpedv X-ME-Proxy: Received: from urd.tobik.me (ip5f5ae20b.dynamic.kabel-deutschland.de [95.90.226.11]) by mail.messagingengine.com (Postfix) with ESMTPA id 4FEA98005A; Sun, 13 Oct 2019 02:27:30 -0400 (EDT) Date: Sun, 13 Oct 2019 08:27:28 +0200 From: Tobias Kortkamp To: Rodrigo Osorio Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r514349 - in head/sysutils: . sanoid sanoid-devel Message-ID: <20191013062728.GC89782@urd.tobik.me> References: <201910121812.x9CICaTk096146@repo.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="4jXrM3lyYWu4nBt5" Content-Disposition: inline In-Reply-To: <201910121812.x9CICaTk096146@repo.freebsd.org> User-Agent: Mutt/1.12.1 (2019-06-15) X-Rspamd-Queue-Id: 46rWvT0lHWz4Zj9 X-Spamd-Bar: - Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-1.81 / 15.00]; local_wl_from(0.00)[freebsd.org]; NEURAL_HAM_MEDIUM(-0.81)[-0.809,0]; ASN(0.00)[asn:11403, ipnet:64.147.123.0/24, country:US]; NEURAL_HAM_LONG(-1.00)[-0.998,0] X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.29 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: Sun, 13 Oct 2019 06:27:33 -0000 --4jXrM3lyYWu4nBt5 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Oct 12, 2019 at 06:12:36PM +0000, Rodrigo Osorio wrote: > Author: rodrigo > Date: Sat Oct 12 18:12:36 2019 > New Revision: 514349 > URL: https://svnweb.freebsd.org/changeset/ports/514349 >=20 > Log: > New ports sysutils/sanoid and sysutils/sanoid-devel > =20 > Sanoid is a policy-driven snapshot management tool > for ZFS filesystems. You can use Sanoid to create, > automatically and monitor snapshots. > =20 > Add sysutils/sanoid-devel to track interesting > moments on the master branch. > =20 > PR: 238584 > Submitted by: > > Added: head/sysutils/sanoid-devel/Makefile > =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 newly added) > +++ head/sysutils/sanoid-devel/Makefile Sat Oct 12 18:12:36 2019 (r514349) > @@ -0,0 +1,51 @@ > +# $FreeBSD$ > + > +PORTNAME=3D sanoid > +PORTVERSION=3D1.0.0.20190619 > +CATEGORIES=3D sysutils > +PKGNAMESUFFIX=3D-devel > + > +MAINTAINER=3D hartzell@alerce.com > +COMMENT=3D Policy-driven snapshot management and replication tools > + > +LICENSE=3D GPLv3 > +LICENSE_FILE=3D ${WRKSRC}/LICENSE > + > +RUN_DEPENDS=3D ${LOCALBASE}/bin/lzop:archivers/lzop \ > + ${LOCALBASE}/bin/mbuffer:misc/mbuffer \ > + ${LOCALBASE}/bin/pv:sysutils/pv \ > + p5-Capture-Tiny>=3D0:devel/p5-Capture-Tiny \ > + p5-Config-IniFiles>=3D0:devel/p5-Config-IniFiles > + > +USES=3D shebangfix perl5 > + > +USE_GITHUB=3D yes > +GH_ACCOUNT=3D jimsalterjrs > +GH_PROJECT=3Dsanoid Redundant. GH_PROJECT is set to ${PORTNAME} by default. > +GH_TAGNAME=3Da90eec3 > + > +USE_PERL5=3D run > +SHEBANG_FILES=3D findoid sanoid sleepymutex syncoid > + > +# line 19: my %args =3D ("configdir" =3D> "/etc/sanoid"); > +post-patch: > + @${REINPLACE_CMD} -i '' -e 's|/etc/sanoid|${PREFIX}/etc/sanoid|' \ > + ${WRKSRC}/sanoid > + @${REINPLACE_CMD} -i '' \ > + -e 's|:/bin:/usr/bin:/sbin|:${PREFIX}/bin:/bin:/usr/bin:/sbin|' \ > + ${WRKSRC}/syncoid > + > +do-build: If the port does not build anything this should be NO_BUILD=3D yes =46rom the looks of it, we could probably also add NO_ARCH=3D yes > + > +do-install: > + ${INSTALL_SCRIPT} ${WRKSRC}/findoid ${STAGEDIR}${PREFIX}/bin > + ${INSTALL_SCRIPT} ${WRKSRC}/sanoid ${STAGEDIR}${PREFIX}/bin > + ${INSTALL_SCRIPT} ${WRKSRC}/sleepymutex ${STAGEDIR}${PREFIX}/bin > + ${INSTALL_SCRIPT} ${WRKSRC}/syncoid ${STAGEDIR}${PREFIX}/bin > + @${MKDIR} ${STAGEDIR}/${ETCDIR} > + ${INSTALL_DATA} ${WRKSRC}/sanoid.defaults.conf \ > + ${STAGEDIR}${ETCDIR}/sanoid.defaults.conf > + ${INSTALL_DATA} ${WRKSRC}/sanoid.conf \ > + ${STAGEDIR}${ETCDIR}/sanoid.conf.sample > + > +.include >=20 > Added: head/sysutils/sanoid-devel/distinfo > =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 newly added) > +++ head/sysutils/sanoid-devel/distinfo Sat Oct 12 18:12:36 2019 (r514349) > @@ -0,0 +1,3 @@ > +TIMESTAMP =3D 1560978182 > +SHA256 (jimsalterjrs-sanoid-1.0.0.20190619-a90eec3_GH0.tar.gz) =3D 32fe6= ef68735e074f8c11d28e06d4e4d9f35cde7b80bacb20636f0fb1fb45500 > +SIZE (jimsalterjrs-sanoid-1.0.0.20190619-a90eec3_GH0.tar.gz) =3D 113654 >=20 > Added: head/sysutils/sanoid-devel/pkg-descr > =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 newly added) > +++ head/sysutils/sanoid-devel/pkg-descr Sat Oct 12 18:12:36 2019 (r51434= 9) > @@ -0,0 +1,4 @@ > +Sanoid is a policy-driven snapshot management tool for ZFS > +filesystems. More prosaically, you can use Sanoid to create, > +automatically thin, and monitor snapshots and pool health from a > +single eminently human-readable TOML config file. Missing WWW here. >=20 > Added: head/sysutils/sanoid-devel/pkg-message > =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 newly added) > +++ head/sysutils/sanoid-devel/pkg-message Sat Oct 12 18:12:36 2019 (r514= 349) > @@ -0,0 +1,28 @@ > +[ > +{ > + message: < +************************************************************************= *** > + > + Sanoid requires additional configuration in the ${PREFIX}/etc/sanoid.co= nf=20 > + file. > + > + A sample has been installed in ${PREFIX}/etc/sanoid.conf.sample and Wrong path to sanoid.conf. It is installed under /usr/local/etc/sanoid/sanoid.conf. > + additional documentation can be found at the sanoid master site: > +=20 > + https://github.com/jimsalterjrs/sanoid The homepage should be in pkg-descr too. > + > +************************************************************************= *** > +EOD > +} > +{ > + message: < +************************************************************************= *** > + > + If you have created a local configuration file, > + ${PREFIX}/etc/sanoid.conf, you will need to remove it separately. > + > +************************************************************************= *** > +EOD > + type: remove > +} > +] Redundant. @sample already prints a message like this on deinstall but only when sanoid.conf has been modified. Please remove the noisy '****' banners. They do not add anything of value. > Added: head/sysutils/sanoid/Makefile > =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 newly added) > +++ head/sysutils/sanoid/Makefile Sat Oct 12 18:12:36 2019 (r514349) > @@ -0,0 +1,51 @@ > +# $FreeBSD$ > + > +PORTNAME=3D sanoid > +DISTVERSION=3D v2.0.1 > +CATEGORIES=3D sysutils Should probably be DISTVERSIONPREFIX=3D v DISTVERSION=3D 2.0.1 > +post-patch: > [...] > + @${REINPLACE_CMD} -i '' \ > + -e 's|/usr/bin/lzop|${PREFIX}/bin/lzop|' \ > + -e 's|/usr/bin/mbuffer|${PREFIX}/bin/mbuffer|' \ > + -e 's|/usr/bin/pv|${PREFIX}/bin/pv|' \ > + -e 's|/usr/bin/sudo|${PREFIX}/bin/sudo|' \ > + ${WRKSRC}/syncoid Dependencies come from LOCALBASE, so this should use LOCALBASE not PREFIX. Most of the items from sanoid-devel also apply to sanoid. --4jXrM3lyYWu4nBt5 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGTBAEBCgB9FiEElXvTEJc6ePgdQuobpPCftzzFH2EFAl2iw9BfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDk1 N0JEMzEwOTczQTc4RjgxRDQyRUExQkE0RjA5RkI3M0NDNTFGNjEACgkQpPCftzzF H2EZNgf+PDElgEOgp8yG82ll4svtk5KKT5Nfe0TEu4vu6fbAEMCeUcgXcOZ6f+FS j/zddTH3DsXaYdSEf27R7KPFiScvBewiUxWTBiQCbifuFI0YaeU9HINFaYaW9k1W uud88ZmOr/mKtg1mC5jQVw0PnR1AFUOIK6/kGuRezBM0m4ef5UkfArh69dfui4T5 YtG0FYmpLOq7FtFPuHTbfQv16oUWsOK7aY6a6z8O6ZXdPzj7dfycCL+gYg6vUBoW WVHrwnI4cQRVHK9LoJ1jokzjinhvaPeQ1wQLTUCEfavY6f3VcgRmI0CLeLYJBARc nbDwShDLNcAGBIzIoimIVa6VPhdiHw== =nmQ0 -----END PGP SIGNATURE----- --4jXrM3lyYWu4nBt5--