From owner-freebsd-python@freebsd.org Fri Jun 30 02:57:01 2017 Return-Path: Delivered-To: freebsd-python@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 BF135D864BB for ; Fri, 30 Jun 2017 02:57:01 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from mailman.ysv.freebsd.org (mailman.ysv.freebsd.org [IPv6:2001:1900:2254:206a::50:5]) by mx1.freebsd.org (Postfix) with ESMTP id A53447D02C for ; Fri, 30 Jun 2017 02:57:01 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: by mailman.ysv.freebsd.org (Postfix) id A1B5AD864BA; Fri, 30 Jun 2017 02:57:01 +0000 (UTC) Delivered-To: python@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 9F87DD864B9 for ; Fri, 30 Jun 2017 02:57:01 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (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 81B917D02A for ; Fri, 30 Jun 2017 02:57:01 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id v5U2v0nf037289 for ; Fri, 30 Jun 2017 02:57:01 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: python@FreeBSD.org Subject: [Bug 219687] [NEW PORT] net/google-compute-engine: User daemon for Google Compute Engine Date: Fri, 30 Jun 2017 02:57:00 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: feature, needs-patch, needs-qa X-Bugzilla-Severity: Affects Some People X-Bugzilla-Who: helen.koike@collabora.com X-Bugzilla-Status: Open X-Bugzilla-Resolution: X-Bugzilla-Priority: Normal X-Bugzilla-Assigned-To: koobs@FreeBSD.org X-Bugzilla-Flags: maintainer-feedback? X-Bugzilla-Changed-Fields: attachments.isobsolete attachments.created Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-python@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: FreeBSD-specific Python issues List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 30 Jun 2017 02:57:01 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D219687 Helen Koike changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #183898|0 |1 is obsolete| | Attachment #183899|0 |1 is obsolete| | --- Comment #10 from Helen Koike --- Created attachment 183934 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D183934&action= =3Dedit Patch which adds google-compute-engine v2.4.0 in freebsd-ports (In reply to Kubilay Kocak from comment #8) > Thank you for your new port contribution Helen. >=20 > Pending further review, initial review items (needing updated patch) are: Thanks for your review I applied most of the items, please see new attached patch. But I still have some questions below. >=20 > - Add missing PKGNAMEPREFIX=3D${PYTHON_PKGNAMEPREFIX} Done > - Missing secondary 'python' virtual category in CATEGORIES Done > - Add USE_PYTHON=3Dconcurrent if the port is multiple python version conc= urrent > installation safe (all files are uniquely named and dont conflict) Done > - Use 'CHEESESHOP' (PyPI) as MASTER_SITES unless there is a compelling re= ason > not to (for instance, test or other important files missing from source > distribution on PyPI) I added CHEESESHOP, but do I need to define MASTER_SITES when USE_GITHUB=3D= yes ? > - Add test support (using do-test: target), adding TEST_DEPENDS (py.test,= etc > as per setup.py/tox.ini) and executing ${PYTHON_CMD} -m pytest or ${PYTHO= N_CMD} > setup.py test if supported. It's OK if not all test dependencies have been > ported yet, just add the ones that have, adding comments for the ones that > havent. setup.py test is not supported, as explained in the previous comment, only = mock tests are supported and I am not sure if it is worthy to port them all. > - setuptools appears (in setup.py) as an install_requires (RUN_DEPENDS) n= ot a > setup_requires (BUILD_DEPENDS) as in the patch. I don't know why its expl= icitly > in install_requires though. hm, I'll check with upstream maintainers, it centenly doesn't use setuptool= s at run time > - Add LICENSE_FILE=3D${WRKSRC}/path/to/license when one exists in the > distribution file Done > - netifaces/netaddr don't appear as setup.py (compulsory) or tox.ini > (potentially optional) dependencies. Can you elaborate on their source and > function? The upstream project doesn't use netifaces/netaddr. For example, it uses /sys/class/net//address to retrieve the mac address. As FreeBSD doesn't seem to have this patch in sysfs, I changed the script to use netifaces/netaddr instead and I added them in the dependencies list >=20 > If possible (highly desirable/recommended), please provide as an attachme= nt the > results of the package test suite as well. As the mock tests are not ported, I am testing by hand so I unfortunatelly doesn't have a test log to attach. >=20 > For more information on creating ports for Python packages, see: >=20 > https://wiki.freebsd.org/Python/PortsPolicy Thanks for this link, I also add the following changes: * I set the PORTVERSION back to 2.4.0 but I had to define GH_TAGNAME and DISTNAME so it downloads the package correctly, is this ok? -PORTVERSION=3D 20170609 ... +PORTVERSION=3D 2.4.0 +DISTNAME=3D ${GH_ACCOUNT}-${GH_PROJECT}-${GH_TAGNAME} ... +GH_TAGNAME=3D 20170609 * About the patches, I know they are not ready to be upstreamable yet (it is not that straight forward to make it compatible to Linux), but I'll work on that. Should I add comments on the patches anyway? * Portlint result: # portlint -AC WARN: Makefile: using hyphen in PORTNAME. consider using PKGNAMEPREFIX and/= or PKGNAMESUFFIX. WARN: /usr/ports/net/google-compute-engine/pkg-message: possible use of absolute pathname "/etc/rc.conf". 0 fatal errors and 2 warnings found. It seems two false positives, the first one wasn't firing before, not sure = why it is appearing now. About the second one, I checked how other packages men= tion /etc/rc.conf and all of them do like this, so I suppose this is the right w= ay to do. Thanks --=20 You are receiving this mail because: You are on the CC list for the bug.=