zsh-workers
 help / color / mirror / code / Atom feed
From: Bart Schaefer <schaefer@brasslantern.com>
To: zsh-workers@sunsite.dk
Subject: PATCH: Re: Failure of Test/V01 in 4.1.0-dev-2
Date: Fri, 28 Sep 2001 09:57:23 +0000	[thread overview]
Message-ID: <1010928095724.ZM16337@candle.brasslantern.com> (raw)
In-Reply-To: <001901c147e7$87af3ab0$21c9ca95@mow.siemens.ru>

On Sep 28, 10:33am, Borsenkow Andrej wrote:
} > } It looks like zsh/zftp doesn't specify a dependency on zsh/net/tcp
} > 
} > Hrm, it does specify the dependency in the .mdd file, but
} 
} We discussed it already. Currently there is no mechanism that allows one
} *external* module to load another *external* module it depends upon.

Ah, right, then the test code should handle that.

However, it appears that "zmodload -u zsh/zftp" dumps core even if the
zsh/net/tcp module is still loaded, so there's some kind of a bug in
zftp.c.

Index: Src/V01zmodload.ztst
===================================================================
diff -c -r1.2 V01zmodload.ztst
--- Src/V01zmodload.ztst	2001/07/10 09:05:25	1.2
+++ Src/V01zmodload.ztst	2001/09/28 09:53:39
@@ -12,6 +12,10 @@
    eval "$name $modfile $link $auto $load"
    [[ $link == no ]] && continue
    mods=($mods $name)
+   moddeps=
+   modfile=$ZTST_srcdir/../$modfile
+   eval ${${${(f)"$(<$modfile)"}[(r)moddeps=*]}:-:}
+   [[ -n $moddeps ]] && zmodload -d $name $=moddeps
  done < $ZTST_testdir/../config.modules
 
  zmodunload() {
@@ -157,5 +161,5 @@
 %clean
 
  eval "$deps"
- unset deps name modfile link auto load funcs mods
+ unset deps name modfile link auto load funcs mods moddeps
  unfunction zmodunload


-- 
Bart Schaefer                                 Brass Lantern Enterprises
http://www.well.com/user/barts              http://www.brasslantern.com

Zsh: http://www.zsh.org | PHPerl Project: http://phperl.sourceforge.net   


  reply	other threads:[~2001-09-28  9:57 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-27 12:39 4.1.0-dev-2 Peter Stephenson
2001-09-27 14:20 ` Failure of Test/V01 in 4.1.0-dev-2 Geoff Wing
2001-09-27 15:07   ` Bart Schaefer
2001-09-27 15:43     ` Bart Schaefer
2001-09-28  6:33       ` Borsenkow Andrej
2001-09-28  9:57         ` Bart Schaefer [this message]
2001-09-28 10:04           ` Core dump unloading zsh/zftp Bart Schaefer
2001-09-28 10:22             ` Peter Stephenson
2001-09-28 10:33               ` Borsenkow Andrej
2001-09-28 11:06                 ` Peter Stephenson
2001-09-28 11:15                   ` Borsenkow Andrej

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1010928095724.ZM16337@candle.brasslantern.com \
    --to=schaefer@brasslantern.com \
    --cc=zsh-workers@sunsite.dk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).