From mboxrd@z Thu Jan 1 00:00:00 1970 From: rekado Subject: Re: [PATCH] Add 'waf-build-system'. Date: Tue, 03 Feb 2015 23:46:13 +0100 Message-ID: <87pp9qsjey.fsf@mango.localdomain> References: <87pp9yvap6.fsf@mango.localdomain> <87386pfiga.fsf@gnu.org> 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]:60361) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YImEk-0001kr-9x for guix-devel@gnu.org; Tue, 03 Feb 2015 17:46:27 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YImEj-00069v-C6 for guix-devel@gnu.org; Tue, 03 Feb 2015 17:46:26 -0500 In-reply-to: <87386pfiga.fsf@gnu.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-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Ludovic =?utf-8?Q?Court=C3=A8s?= Cc: guix-devel@gnu.org Ludovic Courtès writes: > Ricardo Wurmus skribis: > >> Unlike the Python build system, however, there is a configure phase, and >> not everything important happens during the install phase. > > OK. Did you find it more appropriate to extend ‘gnu-build-system’ > rather than ‘python-build-system’? The way I see it the waf-build-system is a descendant of the gnu-build-system in that it has the same phases; it only uses a different tool (namely the waf script). The python-build-system on the other hand concentrates everything in the install "phase". The waf-build-system looks similar to the python-build-system only because the waf script needs to be run with a python interpreter, and it could be either version 2 or 3. This is why I extended the gnu-build-system (as a "parent") and imported the default-python procedures from python-build-system (as a "cousin"). Does this make any sense? > >> From adc52a74fb12943fd77c97bf75a2092e839f9024 Mon Sep 17 00:00:00 2001 >> From: Ricardo Wurmus >> Date: Wed, 28 Jan 2015 22:39:03 +0100 >> Subject: [PATCH] Add 'waf-build-system'. >> >> * guix/build-system/waf.scm, guix/build/waf-build-system.scm: New files. >> * Makefile.am (MODULES): Add them. > > Could you add a few lines to guix.texi, under “Build Systems”? Other > than that this looks good to me. I'll try and send an updated patch. ~~ Ricardo