Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Fri, 31 Jan 2003 14:32:12 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Fri, 31 Jan 2003 14:32:12 -0500 Received: from chaos.physics.uiowa.edu ([128.255.34.189]:10983 "EHLO chaos.physics.uiowa.edu") by vger.kernel.org with ESMTP id ; Fri, 31 Jan 2003 14:32:10 -0500 Date: Fri, 31 Jan 2003 13:41:26 -0600 (CST) From: Kai Germaschewski X-X-Sender: kai@chaos.physics.uiowa.edu To: Pete Zaitcev cc: linux-kernel@vger.kernel.org, Konrad Eisele Subject: Re: Perl in the toolchain In-Reply-To: <20030131133929.A8992@devserv.devel.redhat.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 31 Jan 2003, Pete Zaitcev wrote: > Kai, what's your opinion? I suspect you missed my posing to l-k > Date: Thu, 30 Jan 2003 14:53:59 +0100 > From: Konrad Eisele > To: "PeteZaitcev" > Subject: Adding sparc-leon linux to sourcetree > > ..... > There is also one change I have made on the buildsystem. Because I'm using some > perl inline scripts in the $cmd_xxx the >'< and >$< signs in the inline perl scripts > cause trouble (perl -e '...$x=....'), the >'< because of the echo command, the >$< when > rereading from the xxx..cmd files. Could this be applied to the original file? Unfortunately, I cannot find the original posting quoted above, since that would probably reveal where the actual usage of perl is. Generally, we've been trying to not make perl a prequisite for the kernel build, and I'd like to keep it that way. Except for some arch specific stuff I don't really care about, the uses of perl are for the optional "make checkconfig" etc. (which btw look mostly obsolete and should probably be killed), and for generating some firmware, though by default a shipped version of the generated files is used. That said, I'm not necessarily opposed to allowing for use of ',$ in the command variables as the change above does (getting that as a patch would be nice, though). --Kai > scripts/Makefile.lib: > +-------------------------------+ > > # =========================================================================== > # Generic stuff > # =========================================================================== > > # function to only execute the passed command if necessary > # the ' -> '\'' and $ to $$ substitution are done if $(cmd_$(1)) includes a inline perlscript > > if_changed = $(if $(strip $? \ > $(filter-out $(cmd_$(1)),$(cmd_$@))\ > $(filter-out $(cmd_$@),$(cmd_$(1)))),\ > @set -e; \ > $(if $($(quiet)cmd_$(1)),echo ' $(subst ','\'',$($(quiet)cmd_$(1)))';) \ > $(cmd_$(1)); \ > echo 'cmd_$@ := $(subst $$,$$$$,$(subst ','\'',$(cmd_$(1))))' > $(@D)/.$(@F).cmd) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/