From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark H Weaver Subject: Re: [PATCH] gnu: Add quvi. Date: Fri, 06 Mar 2015 18:32:54 -0500 Message-ID: <87385hln15.fsf@netris.org> References: <87385ke82v.fsf@taylan.uni.cx> <20150306215436.GA32174@debian> <877futlp4x.fsf@netris.org> <20150306230539.GA30428@debian> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:47741) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YU1js-0007DS-9C for guix-devel@gnu.org; Fri, 06 Mar 2015 18:33:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YU1jd-0000hL-72 for guix-devel@gnu.org; Fri, 06 Mar 2015 18:33:04 -0500 Received: from world.peace.net ([50.252.239.5]:55467) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YU1jd-0000hB-3W for guix-devel@gnu.org; Fri, 06 Mar 2015 18:32:49 -0500 In-Reply-To: <20150306230539.GA30428@debian> (Andreas Enge's message of "Sat, 7 Mar 2015 00:05:39 +0100") 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: Andreas Enge Cc: guix-devel@gnu.org Andreas Enge writes: > On Fri, Mar 06, 2015 at 05:47:26PM -0500, Mark H Weaver wrote: >> I'm familiar with these packages, and I tend to agree that they are so >> closely tied that it's reasonable to commit them together. > > That is not common practice so far, no? I do not see what would be > the problem when committing them separately. I don't feel strongly either way. >> > Your indentation (here and further down) is not quite consistens, sometimes >> > it it one, sometimes two spaces. >> He's using the auto-indenter of Emacs Scheme Mode, which faithfully >> follows the rules in our .dir-locals.el file. > > Curious. Why does it use sometimes one, sometimes two spaces? Can this be > explained so that non-emacs users follow the same style? I think we're trying to be tolerant of some inconsistency in indentation, but the usual rule is that continuation lines within a parenthesized list are lined up with the first argument (second list item) on the first line, or if the first line only contains one list item, then they are all lined up. However, if there's an entry in .dir-locals.el, then it is considered a special form and is indented two spaces. >> >> + (string-append "liblua_LIBS=-L" lua "/libs -llua"))))) >> > These are two flags and should be two entries in the list. >> What Taylan wrote above looks right to me. The -L[...] and -llua are >> two flags, but the variable setting "liblua_LIBS=-L[...]/libs -llua" >> needs to be one argument to configure. > > I do not get this. Why should two flags be one argument? Do you mean that > on the command line, one would write: > ./configure "liblua_LIBS=-L/gnu/store/.../libs -llua" Yes. > and that the call of system* adds these quotation marks implicitly? It doesn't add quotation marks implicitly, but it does the same thing that the shell would do if quotation marks were added, namely that the entire quoted item is passed as a single argument to ./configure. If they were instead passed as two arguments, as would be done with the command ./configure liblua_LIBS=-L/gnu/store/.../libs -llua Then the 'liblua_LIBS' variable would be "-L/gnu/store/.../libs" and "-llua" would be passed as a separate argument to configure, which certainly wouldn't do what we want and might raise an error. Mark