On 20:39 Sat 10 Jul , Denys Vlasenko wrote: > On Fri, Jul 9, 2010 at 9:46 AM, Natanael Copa <[email protected]> wrote: > > On Fri, Jul 9, 2010 at 9:23 AM, Natanael Copa <[email protected]> > > wrote: > >> This testcase triggers bug in diff when a trailing / is used on dir. > >> > >> Bug was reported as https://bugs.busybox.net/show_bug.cgi?id=2203 > > > > while testing a possible fix for the above bug i noticed that if you > > do 'diff -r / /tmp' I get some interesting results. Seems like diff > > follwos symlinks with some interesting results: > > > > diff: > > /proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/proc/self/task/13771/fd/3/home/ncopa/aports/testing/py-application/src/python-application-1.1.5.tar.gz: > > Too many levels of symbolic links > > > > > > diffutils' diff does not seem to recurse dirs that don't exist in both > > dirs so it finishes dead fast while busybox diff takes forever. > > Fixed in git.
Ops, somehow I missed this bug report. Yeah that fix is ok, but it's an ugly hack unfortunately :/ It's probably the way which results in the smallest code though. I will give it some thought after the weekend. _______________________________________________ busybox mailing list [email protected] http://lists.busybox.net/mailman/listinfo/busybox
