From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH v2 03/10] gnu: Add python-pytest-pep8. Date: Mon, 25 Apr 2016 14:19:03 -0400 Message-ID: <20160425181903.GE8957@jasmine> References: <1461244921-7412-1-git-send-email-donttrustben@gmail.com> <1461503184-8841-1-git-send-email-donttrustben@gmail.com> <1461503184-8841-4-git-send-email-donttrustben@gmail.com> <20160424170405.GA12480@debian-netbook> <571D5AA1.9090605@gmail.com> <571E0F66.2070703@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:52679) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aul6C-0004JO-4c for Guix-devel@gnu.org; Mon, 25 Apr 2016 14:19:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aul68-0000fw-U3 for Guix-devel@gnu.org; Mon, 25 Apr 2016 14:19:08 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:59267) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aul68-0000fd-QQ for Guix-devel@gnu.org; Mon, 25 Apr 2016 14:19:04 -0400 Content-Disposition: inline In-Reply-To: <571E0F66.2070703@gmail.com> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Ben Woodcroft Cc: Guix-devel@gnu.org On Mon, Apr 25, 2016 at 10:36:54PM +1000, Ben Woodcroft wrote: > On 25/04/16 18:43, Ricardo Wurmus wrote: > >Ben Woodcroft writes: > > > >>>>+ (propagated-inputs > >>>>+ `(("python-pytest-cache", python-pytest-cache) > >>>>+ ("python-pytest" ,python-pytest) > >>>>+ ("python-pep8" ,python-pep8))) > >>>Are you sure all of these need to be propagated? > >>That's what the install_requires in setup.py is telling me. > >Doesn’t this mean that these packages are only needed at build time? > >Could they not be made native-inputs then? > | > I don't believe so. > > http://pythonhosted.org/setuptools/setuptools.html#new-and-changed-setup-keywords > > >install_requires| > > A string or list of strings specifying what other distributions need to be installed when this one is. This seems like an unusual case, since the package in question appears to be a build-time tool. So, it would make sense to have things like python-pytest available during python-pytest-pep8's run-time. If we have a use case for python-pytest-pep8, it could be worthwhile to see if it will work without propagating these anyways; I've noticed some Python packages have a way to link to non-propagated-inputs.