Re: gnu-web-doc-update: fails to add new directories

[Pádraig Brady]

On 15/12/12 20:41, Jim Meyering wrote:
> Akim Demaille wrote:
>> Le 13 déc. 2012 à 18:06, Jim Meyering <address@hidden> a écrit :
>>
>>>> cvsu does not report that there are directories and files inside the
>>>> 'D's.  Is it wrong to simply cvs add everything that is there?
>>>> Just skipping CVS, but adding all the rest using "find"?
>>>> Say "find . -name CVS -prune -o -print".
>>>
>>> That sounds like it will work.
>>> Thanks again.
>>
>> Here is my proposal.  I don't see anything ready to _remove_ bits
>> that are no longer generated.  This is a problem, since for instance
> 
> Hi Akim!
> Yes, it'd be good to fix that, someday.

FYI I fixed that a while ago with:
http://git.sv.gnu.org/gitweb/?p=gnulib.git;a=commit;h=9a417cf

>> commit a1d1f4b7fc9b46fc37a8dc55b591768a80ead566
>> Author: Akim Demaille <address@hidden>
>> Date:   Fri Dec 14 12:58:35 2012 +0100
>>
>>     gnu-web-doc-update: add all the new files, even in new directories
>>
>>     See http://lists.gnu.org/archive/html/bug-gnulib/2012-12/msg00057.html
>>
>>     * build-aux/gnu-web-doc-update (--dry-run, $dryrun): New.
>>     Use it.
>>     (main): Don't use cvsutils to get the list of unknown files,
>>     just add all the existing files and directories.

That change is causing an issue in coreutils wrt empty dirs,
which causes new files to be missed due to:
  cvs [add aborted]: there is a version in ./ps already
  ...

I've installed the attached change to add dirs separately and individually,
as cvs seems to have different treatment for files and dirs.

thanks,
Pádraig.

web-doc-update-new-files.patch
Description: Text Data