From owner-svn-ports-head@freebsd.org Sun Apr 30 14:26:43 2017 Return-Path: Delivered-To: svn-ports-head@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 4B569D57566; Sun, 30 Apr 2017 14:26:43 +0000 (UTC) (envelope-from cpm@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (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 1DF0540E; Sun, 30 Apr 2017 14:26:43 +0000 (UTC) (envelope-from cpm@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id v3UEQfxb001158; Sun, 30 Apr 2017 14:26:41 GMT (envelope-from cpm@FreeBSD.org) Received: (from cpm@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id v3UEQfhT001154; Sun, 30 Apr 2017 14:26:41 GMT (envelope-from cpm@FreeBSD.org) Message-Id: <201704301426.v3UEQfhT001154@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: cpm set sender to cpm@FreeBSD.org using -f From: "Carlos J. Puga Medina" Date: Sun, 30 Apr 2017 14:26:41 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r439825 - in head/security: py-acme py-acme/files py-certbot py-certbot/files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.23 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: Sun, 30 Apr 2017 14:26:43 -0000 Author: cpm Date: Sun Apr 30 14:26:41 2017 New Revision: 439825 URL: https://svnweb.freebsd.org/changeset/ports/439825 Log: security/py-{acme,certbot}: Make argparse a conditional dependency Backport an upstream patch [1][2] to setup.py making argparse a conditional dependency. Without this change, the following error is observed at run-time: pkg_resources.DistributionNotFound: The 'argparse' distribution was not found and is required by certbot This is due to the separate argparse distribution (via PyPI) no longer being required for Python versions >= 2.7 and >= 3.2, as they contain the argparse module in the standard library. Since FreeBSD only supports Python versions with the argparse module built in, *and* having removed the devel/py-argparse port, the install_requires=argparse dependency cannot be satisfied, as install_requires is only relevant for non standard library (PyPI) packages/dependencies. [1] https://github.com/certbot/certbot/pull/4554 [2] https://github.com/certbot/certbot/issues/4485 PR: 218973 Reported by: olgeni Approved by: koobs (py-certbot, maintainer) Obtained from: https://github.com/certbot/certbot/commit/89af460792fcdfb23c7dc4f9fcdec1bfa07a2656 Differential Revision: https://reviews.freebsd.org/D10546 Added: head/security/py-acme/files/ head/security/py-acme/files/patch-setup.py (contents, props changed) head/security/py-certbot/files/ head/security/py-certbot/files/patch-setup.py (contents, props changed) Modified: head/security/py-acme/Makefile head/security/py-certbot/Makefile Modified: head/security/py-acme/Makefile ============================================================================== --- head/security/py-acme/Makefile Sun Apr 30 14:19:50 2017 (r439824) +++ head/security/py-acme/Makefile Sun Apr 30 14:26:41 2017 (r439825) @@ -3,6 +3,7 @@ PORTNAME= acme PORTVERSION= 0.13.0 +PORTREVISION= 1 PORTEPOCH= 1 CATEGORIES= security python MASTER_SITES= CHEESESHOP Added: head/security/py-acme/files/patch-setup.py ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/security/py-acme/files/patch-setup.py Sun Apr 30 14:26:41 2017 (r439825) @@ -0,0 +1,21 @@ +--- setup.py.orig 2017-04-30 10:29:55 UTC ++++ setup.py +@@ -8,7 +8,6 @@ version = '0.13.0' + + # Please update tox.ini when modifying dependency version requirements + install_requires = [ +- 'argparse', + # load_pem_private/public_key (>=0.6) + # rsa_recover_prime_factors (>=0.8) + 'cryptography>=0.8', +@@ -28,6 +27,10 @@ install_requires = [ + 'six', + ] + ++# env markers cause problems with older pip and setuptools ++if sys.version_info < (2, 7): ++ install_requires.append('argparse') ++ + dev_extras = [ + 'nose', + 'tox', Modified: head/security/py-certbot/Makefile ============================================================================== --- head/security/py-certbot/Makefile Sun Apr 30 14:19:50 2017 (r439824) +++ head/security/py-certbot/Makefile Sun Apr 30 14:26:41 2017 (r439825) @@ -3,6 +3,7 @@ PORTNAME= certbot PORTVERSION= 0.13.0 +PORTREVISION= 1 PORTEPOCH= 1 CATEGORIES= security python MASTER_SITES= CHEESESHOP Added: head/security/py-certbot/files/patch-setup.py ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/security/py-certbot/files/patch-setup.py Sun Apr 30 14:26:41 2017 (r439825) @@ -0,0 +1,21 @@ +--- setup.py.orig 2017-04-30 10:33:38 UTC ++++ setup.py +@@ -36,7 +36,6 @@ version = meta['version'] + # https://github.com/pypa/pip/issues/988 for more info. + install_requires = [ + 'acme=={0}'.format(version), +- 'argparse', + # We technically need ConfigArgParse 0.10.0 for Python 2.6 support, but + # saying so here causes a runtime error against our temporary fork of 0.9.3 + # in which we added 2.6 support (see #2243), so we relax the requirement. +@@ -56,6 +55,10 @@ install_requires = [ + 'zope.interface', + ] + ++# env markers cause problems with older pip and setuptools ++if sys.version_info < (2, 7): ++ install_requires.append('argparse') ++ + dev_extras = [ + # Pin astroid==1.3.5, pylint==1.4.2 as a workaround for #289 + 'astroid==1.3.5',