From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7786 invoked from network); 13 Mar 2008 20:28:38 -0000 X-Spam-Checker-Version: SpamAssassin 3.2.4 (2008-01-01) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,URI_NOVOWEL autolearn=no version=3.2.4 Received: from news.dotsrc.org (HELO a.mx.sunsite.dk) (130.225.247.88) by ns1.primenet.com.au with SMTP; 13 Mar 2008 20:28:38 -0000 Received-SPF: none (ns1.primenet.com.au: domain at sunsite.dk does not designate permitted sender hosts) Received: (qmail 57400 invoked from network); 13 Mar 2008 20:28:32 -0000 Received: from sunsite.dk (130.225.247.90) by a.mx.sunsite.dk with SMTP; 13 Mar 2008 20:28:32 -0000 Received: (qmail 4888 invoked by alias); 13 Mar 2008 20:28:28 -0000 Mailing-List: contact zsh-workers-help@sunsite.dk; run by ezmlm Precedence: bulk X-No-Archive: yes X-Seq: 24706 Received: (qmail 4875 invoked from network); 13 Mar 2008 20:28:28 -0000 Received: from bifrost.dotsrc.org (130.225.254.106) by sunsite.dk with SMTP; 13 Mar 2008 20:28:28 -0000 Received: from mta-2.ms.rz.rwth-aachen.de (mta-2.ms.rz.RWTH-Aachen.DE [134.130.7.73]) by bifrost.dotsrc.org (Postfix) with ESMTP id 321D780307F7 for ; Thu, 13 Mar 2008 21:28:19 +0100 (CET) Received: from ironport-out-1.rz.rwth-aachen.de ([134.130.3.58]) by mta-2.ms.rz.RWTH-Aachen.de (Sun Java System Messaging Server 6.2-8.04 (built Feb 28 2007)) with ESMTP id <0JXO003LGRJ7SU10@mta-2.ms.rz.RWTH-Aachen.de> for zsh-workers@sunsite.dk; Thu, 13 Mar 2008 21:28:19 +0100 (CET) Received: from relay.rwth-aachen.de ([134.130.3.1]) by ironport-in-1.rz.rwth-aachen.de with ESMTP; Thu, 13 Mar 2008 21:28:19 +0100 Received: from fsst.voodoo.lan ([212.117.84.203]) by relay.rwth-aachen.de (8.13.7/8.13.3/1) with ESMTP id m2DKSIDX007961 for ; Thu, 13 Mar 2008 21:28:18 +0100 (MET) Received: from hawk by fsst.voodoo.lan with local (Exim 4.69) (envelope-from ) id 1JZu1l-0000Wv-QL for zsh-workers@sunsite.dk; Thu, 13 Mar 2008 21:27:49 +0100 Date: Thu, 13 Mar 2008 21:27:48 +0100 From: Frank Terbeck Subject: zkbd fails: must be run as a function or shell script, not sourced To: zsh workers Mail-followup-to: zsh workers Message-id: <20080313202748.GV30518@fsst.voodoo.lan> MIME-version: 1.0 Content-type: text/plain; charset=iso-8859-1 Content-transfer-encoding: 7BIT Content-disposition: inline X-IronPort-AV: E=Sophos;i="4.25,495,1199660400"; d="scan'208";a="56086940" User-Agent: Mutt/1.5.17 (2008-01-15) X-Virus-Scanned: ClamAV 0.91.2/6225/Thu Mar 13 15:52:37 2008 on bifrost X-Virus-Status: Clean Hi workers, The zkbd function currently fails to start. In particular, it seems to be the following test (I don't actually understand how that test works at all :-): [snip] [[ -o interactive ]] && { local -i ARGC (ARGC=0) 2>/dev/null || { print -u2 ${0}: must be run as a function or shell script, not sourced return 1 } } [snap] Since this worked before, I git-bisected my way through the younger zsh code history and I was able to find the commit that breaks it for me. The last working commit that works is this: 2007-05-28 13:08 pws 23478: crash in reverse-menu-complete if no completion And the next commit (the one that breaks it) is: 2007-05-28 22:57 pws see 23479: add initial features support for modules Since these are commits in a cvs->git conversion of the zsh tree, here is some extra information to locate the change I'm referring to (in case the conversion is not accurate enough): The commit adds this Changelog entry: [snip] * Not posted but see 23479:Completion/Unix/Command/_chmod, Completion/Unix/Command/_chown, Completion/Unix/Command/_cvs, Completion/Unix/Type/_list_files, Completion/Zsh/Command/_stat, Completion/Zsh/Command/_zstyle, Completion/Zsh/Type/_file_descriptors,Doc/Zsh/builtins.yo, Doc/Zsh/compsys.yo, Doc/Zsh/mod_stat.yo, Etc/zsh-development-guide, Src/builtin.c, Src/cond.c, Src/exec.c Src/init.c, Src/mkbltnmlst.sh, Src/mkmakemod.sh, Src/modentry.c, Src/module.c, Src/params.c, Src/zsh.h, Src/Builtins/rlimits.c, Src/Builtins/sched.c, Src/Modules/cap.c, Src/Modules/clone.c, Src/Modules/datetime.c, Src/Modules/example.c, Src/Modules/files.c, Src/Modules/langinfo.c, Src/Modules/langinfo.mdd, Src/Modules/mapfile.c, Src/Modules/mathfunc.c, Src/Modules/parameter.c, Src/Modules/pcre.c, Src/Modules/regex.c, Src/Modules/socket.c, Src/Modules/stat.c, Src/Modules/system.c, Src/Modules/tcp.c, Src/Modules/termcap.c, Src/Modules/terminfo.c, Src/Modules/zftp.c, Src/Modules/zprof.c, Src/Modules/zpty.c, Src/Modules/zselect.c, Src/Modules/zutil.c, Src/Zle/compctl.c, Src/Zle/complete.c, Src/Zle/complist.c, Src/Zle/computil.c, Src/Zle/deltochar.c, Src/Zle/zle_main.c, Src/Zle/zle_thingy.c, Src/Zle/zle_tricky.c, Src/Zle/zleparameter.c, Test/B02typeset.ztst, Test/V01zmodload.ztst, Test/V04features.ztst: add "zmodload -F" and internal features support for modules. [snap] Unfortunately, the commit is rather big, so I won't be able to find the problematic part on my own. I did create a diff between the two commits in question and uploaded it to: I hope that helps. Regards, Frank -- In protocol design, perfection has been reached not when there is nothing left to add, but when there is nothing left to take away. -- RFC 1925