PATCH: Re: Failure of Test/V01 in 4.1.0-dev-2

[Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx>]

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