Re: bugs in dirname module - coreutils portion

[Jim Meyering]

Eric Blake <address@hidden> wrote:
...
> OK, I will do better at not combining whitespace cleanup with real
> patches.  And I agree with your comment that using diff -b as a crutch is
> not wise, so here is the updated patch with no whitespace filtering.

Thanks for the big patch -- and for your patience.

I've been holding off because I don't want this to interfere with
the pending patch to add ACL support.  Since I've made Andreas wait
for so long (it's been years), the least I can do is to defer a
patch like this, so as not to make his job any harder.
It might be one week more.

One question: why continue to use base_name in rm.c, rather than
converting that use to last_component, the way you did in all of
the other cases?

...
> Index: src/rm.c
> ===================================================================
> RCS file: /cvsroot/coreutils/coreutils/src/rm.c,v
> retrieving revision 1.136
> diff -u -p -r1.136 rm.c
> --- src/rm.c  2 Nov 2005 21:53:20 -0000       1.136
> +++ src/rm.c  25 Nov 2005 15:00:21 -0000
> @@ -169,6 +169,7 @@ the contents of that file.  If you want
>  truly unrecoverable, consider using shred.\n\
>  "), stdout);
>        printf (_("\nReport bugs to <%s>.\n"), PACKAGE_BUGREPORT);
> +      free (base);
>      }
>    exit (status);
>  }