[PATCH] patch for lin need_source function

Auke Kok sofar at lunar-linux.org
Wed Apr 14 00:42:38 GMT 2004


Jasper Huijsmans wrote:
> ha, changing just one line seems to be enough as well ;-) See attached.

I'll prolly incorporate this modification with an extra source tarball 
check in lin_module() itself as backup too: reason:

* it is fairly safe to assume all sub-dependencies of a dependency 
already are installed if the dependency itself is already installed

* and if not this is a rare exception that the backup source tarball 
check will catch

* the modification is very effective at eliminating most of the tree 
that it's traversing (it's fast).

thanks for thinking along on this one ;^) expect to patch theedge 
tomorrow...

sofar

> Index: lin
> ===================================================================
> RCS file: /var/cvs/lunar/theedge/sbin/lin,v
> retrieving revision 1.76
> diff -u -r1.76 lin
> --- lin	21 Mar 2004 20:51:44 -0000	1.76
> +++ lin	13 Apr 2004 21:16:29 -0000
> @@ -108,8 +108,8 @@
>        verbose_msg "Spawning download manager"
>        need_source() {(
>          for DEP in $(grep ^$1: $DEPENDS_STATUS | grep :on: | cut -d: -f2) ; do
> -	need_source $DEP
>  	  if ! module_installed $DEP ; then 
> +	    need_source $DEP
>  	    echo $DEP
>  	  fi
>  	done
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Lunar-dev mailing list
> Lunar-dev at lunar-linux.org
> http://dbguin.lunar-linux.org/mailman/listinfo/lunar-dev



More information about the Lunar-dev mailing list