Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 15 Sep 2014 03:50:31 -0400
From:      "Jason E. Hale" <jhale@freebsd.org>
To:        "Vanilla I. Shu" <vanilla@freebsd.org>
Cc:        "svn-ports-head@freebsd.org" <svn-ports-head@freebsd.org>, "svn-ports-all@freebsd.org" <svn-ports-all@freebsd.org>, jadm@dachev.info, ports-committers <ports-committers@freebsd.org>
Subject:   Re: svn commit: r368241 - in head/sysutils: . jadm jadm/files
Message-ID:  <CAJE75NE1p_mf4xHADQgoeKnmk_E7Wej9pfTsGJnPsJOj4PHWcw@mail.gmail.com>
In-Reply-To: <201409150227.s8F2ReuH083121@svn.freebsd.org>
References:  <201409150227.s8F2ReuH083121@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Sep 14, 2014 at 10:27 PM, Vanilla I. Shu <vanilla@freebsd.org> wrote:
> Author: vanilla
> Date: Mon Sep 15 02:27:39 2014
> New Revision: 368241
> URL: http://svnweb.freebsd.org/changeset/ports/368241
> QAT: https://qat.redports.org/buildarchive/r368241/
>
> Log:
>   Add jadm 1.0, freeBSD jail framework with zfs, vnet and jail.conf
>   support.
>
>   PR:           ports/193574
>   Submitted by: Nikolay Dachev <jadm dachev info>
>
> Added:
>   head/sysutils/jadm/
>   head/sysutils/jadm/Makefile   (contents, props changed)
>   head/sysutils/jadm/distinfo   (contents, props changed)
>   head/sysutils/jadm/files/
>   head/sysutils/jadm/files/patch-setup.py   (contents, props changed)
>   head/sysutils/jadm/pkg-descr   (contents, props changed)
> Modified:
>   head/sysutils/Makefile
>
> Modified: head/sysutils/Makefile
> ==============================================================================
> --- head/sysutils/Makefile      Mon Sep 15 02:20:19 2014        (r368240)
> +++ head/sysutils/Makefile      Mon Sep 15 02:27:39 2014        (r368241)
> @@ -394,6 +394,7 @@
>      SUBDIR += isomaster
>      SUBDIR += isomd5sum
>      SUBDIR += istatd
> +    SUBDIR += jadm
>      SUBDIR += jail-primer
>      SUBDIR += jail2
>      SUBDIR += jailctl
>
> Added: head/sysutils/jadm/Makefile
> ==============================================================================
> --- /dev/null   00:00:00 1970   (empty, because file is newly added)
> +++ head/sysutils/jadm/Makefile Mon Sep 15 02:27:39 2014        (r368241)
> @@ -0,0 +1,33 @@
> +# Created by: Nikolay Dachev <jadm@dachev.info>
> +# $FreeBSD$
> +
> +PORTNAME=      jadm
> +PORTVERSION=   1.0
> +CATEGORIES=    sysutils python
> +MASTER_SITES=  CHEESESHOP
> +PKGNAMEPREFIX= ${PYTHON_PKGNAMEPREFIX}
> +
> +MAINTAINER=    jadm@dachev.info
> +COMMENT=       FreeBSD jail framework with zfs, vnet and jail.conf support
> +
> +LICENSE=       BSD3CLAUSE

LICENSE_FILE?

> +
> +BUILD_DEPENDS= ${PYTHON_LIBDIR}/site-packages/paramiko/:${PORTSDIR}/security/py-paramiko \
> +               ${PYTHON_LIBDIR}/site-packages/netifaces.so:${PORTSDIR}/net/py-netifaces \
> +               ${PYTHON_LIBDIR}/site-packages/tabulate.py:${PORTSDIR}/devel/py-tabulate \
> +               ${PYTHON_LIBDIR}/site-packages/ipaddress.py:${PORTSDIR}/net/py-ipaddress

This is a deprecated style of depending on other Python ports.
Format should be:
BUILD_DEPENDS= ${PYTHON_PKGNAMEPREFIX}paramiko>=0:${PORTSDIR}/security/py-paramiko

> +
> +USES=          python
> +USE_PYTHON=    distutils autoplist
> +PLIST_FILES=   man/man8/jadm.8.gz
> +
> +.include <bsd.port.pre.mk>
> +
> +.if ${OSVERSION} < 901000
> +IGNORE=                 freeBSD before 9.1 is not supported
> +.endif

FreeBSD maybe?  OPSYS should also be tested for if you are testing for
OSVERSION.

> +
> +pre-install:
> +       ${CP} ${WRKSRC}/man8/jadm.8 ${WRKDIR}/stage${PREFIX}/man/man8
> +

Yuck.  How about:
post-install:
        ${INSTALL_MAN} ${WRKSRC}/man8/jadm.8 ${STAGEDIR}${MAN8PREFIX}/man/man8


> +.include <bsd.port.post.mk>
>
> Added: head/sysutils/jadm/distinfo
> ==============================================================================
> --- /dev/null   00:00:00 1970   (empty, because file is newly added)
> +++ head/sysutils/jadm/distinfo Mon Sep 15 02:27:39 2014        (r368241)
> @@ -0,0 +1,2 @@
> +SHA256 (jadm-1.0.tar.gz) = 2c654494feb8d0bf29dc3ebc0f8e0d924c99c9168a62f7edc3a672089612c828
> +SIZE (jadm-1.0.tar.gz) = 33335
>
> Added: head/sysutils/jadm/files/patch-setup.py
> ==============================================================================
> --- /dev/null   00:00:00 1970   (empty, because file is newly added)
> +++ head/sysutils/jadm/files/patch-setup.py     Mon Sep 15 02:27:39 2014        (r368241)
> @@ -0,0 +1,10 @@
> +--- setup.py.orig      2014-09-14 17:02:59.000000000 +0800
> ++++ setup.py   2014-09-15 10:21:15.776638606 +0800
> +@@ -37,5 +37,5 @@ setup(
> + )
> +
> + # man page install
> +-if sys.argv[1] == "install":
> +-   os.system('cp ./man8/jadm.8 /usr/local/man/man8/jadm.8')
> ++#if sys.argv[1] == "install":
> ++   #os.system('cp ./man8/jadm.8 /usr/local/man/man8/jadm.8')
>
> Added: head/sysutils/jadm/pkg-descr
> ==============================================================================
> --- /dev/null   00:00:00 1970   (empty, because file is newly added)
> +++ head/sysutils/jadm/pkg-descr        Mon Sep 15 02:27:39 2014        (r368241)
> @@ -0,0 +1,2 @@
> +Jadm is python script which pars /etc/jail.conf in his own format.
> +Use zfs for jail home also bridge interface and epair interfaces for jail vnet.
>

No WWW?



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJE75NE1p_mf4xHADQgoeKnmk_E7Wej9pfTsGJnPsJOj4PHWcw>