> -----Original Message----- > From: Jim Meyering [mailto:j...@meyering.net] > Sent: Friday, July 20, 2012 4:07 PM > To: Joachim Schmitz > Cc: 'Paul Eggert'; bug-gnulib@gnu.org; nagendra...@hp.com; > 10...@debbugs.gnu.org; 'Schmitz, Joachim'; 'Eric Blake' > Subject: Re: bug#10305: coreutils-8.14, "rm -r" fails with EBADF > > Joachim Schmitz wrote: > > >> From: Jim Meyering [mailto:j...@meyering.net] > >> Sent: Friday, July 20, 2012 3:48 PM > >> To: Joachim Schmitz > >> Cc: 'Paul Eggert'; 10...@debbugs.gnu.org; bug-gnulib@gnu.org; 'Eric > > Blake'; > >> 'Schmitz, Joachim'; nagendra...@hp.com > >> Subject: Re: bug#10305: coreutils-8.14, "rm -r" fails with EBADF > >> > >> Joachim Schmitz wrote: > >> ... > >> > I just saw that my patch removed 2 functions more than your's, mine > >> > also removes cache_stat_ok() and is_nondir_lstat(). > >> > Intention? Used where? > >> > >> Hah! I should have temporarily defined-away "inline" to be sure I'd > > removed > >> all of them -- then gcc warns about each defined-but-not-used function. > > I've > >> just done that. Those two in remove.c are the only ones in src/. > >> > >> As penance, I'll do this one for you, too. > > > > ;-) Thanks! > > > > Is anyone looking into my other patches too? > > I don't know. > > However, note that you're using an old version of coreutils. In the latest, su.c > has been removed so you can drop the patches to that file.
Huh? 8.17 is old? It's the latest I could get and still has src/su.c. Anyway, that patch isn't needed for NonStop anyway, as we need an entirely different authentication method. > If no one gets back to you on the other coreutils-specific pieces in two or three > weeks, please feel free to ping us at coreut...@gnu.org. OK, I'll check, after my vacation.
