From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH v3] gnu: Add python-prompt-toolkit, python2-prompt-toolkit. Date: Wed, 29 Jun 2016 22:23:32 -0400 Message-ID: <20160630022332.GA6667@jasmine> References: <20160628101945.4225c7a5@scratchpost.org> <20160629094040.01b99f6a@scratchpost.org> <20160629113033.5297f1e0@scratchpost.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:51768) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bIRe0-0003hF-0Q for guix-devel@gnu.org; Wed, 29 Jun 2016 22:23:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bIRdw-0004uR-GC for guix-devel@gnu.org; Wed, 29 Jun 2016 22:23:55 -0400 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:40178) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bIRdv-0004n0-3U for guix-devel@gnu.org; Wed, 29 Jun 2016 22:23:52 -0400 Content-Disposition: inline In-Reply-To: <20160629113033.5297f1e0@scratchpost.org> 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: Danny Milosavljevic Cc: guix-devel@gnu.org On Wed, Jun 29, 2016 at 11:30:33AM +0200, Danny Milosavljevic wrote: > On Wed, 29 Jun 2016 09:40:40 +0200 > Danny Milosavljevic wrote: > > > +(define-public python2-prompt-toolkit > > + (let ((base (package-with-python2 (strip-python2-variant python-prompt-toolkit)))) > > + (package > > + (inherit base) > > + (native-inputs > > + `(("python2-setuptools" ,python2-setuptools) > > + ,@(package-native-inputs base)))))) > > > Hmm. Now that I made it a native input instead of propagated input, the python2-wcwidth that is compiled alongside python2-prompt-toolkit doesn't find setuptools anymore. Why? In python-wcwidth and python-prompt-toolkit, the 'properties' field that makes the python2-variant system work is missing. I missed that in my review of python-wcwidth. I'll push a follow up to that commit, and then push this patch with the properties field added as well.