On Wednesday 18 June 2008 15:15, George Boudreau wrote: > Hi, > While building busybox-20080618, with defconfig, I stumbled upon a > problem with 'ls'. > > ./busybox ls; echo $? > 1 > ./busybox ls qwert; echo $? > 0 > > Looks like the status logic is inverted. > > if (ENABLE_FEATURE_CLEAN_UP) > dfree(dnp, nfiles); > return (exit_failure == 0); > > should be > > return (exit_failure == 1); > or > return (exit_failure);
Thanks for catching this. I renamed the variable to exit_code now, and fixed the logic. -- vda _______________________________________________ busybox mailing list [email protected] http://busybox.net/cgi-bin/mailman/listinfo/busybox
