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

[Akim Demaille]

Hi Pádraig!

> Le 20 janv. 2016 à 15:16, Pádraig Brady <address@hidden> a écrit :
> 
> 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

Thanks for letting me know!

Wow…

+  { find . \( -name CVS -o -type f -name '.*' \) -prune -o -type f -print
+    (cd "$builddir"/doc/manual/ && find . -type f -print | sed p)
+  } | sort | uniq -u \
+    | $XARGS --no-run-if-empty -- ${rm_stale:-$dryrun} $CVS remove -f

the ‘sed p | uniq -u’ sequence is super smart :)  I guess you had a lot of fun 
finding it :)

> 
>>> 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.

Great!

I wish all this were useless, and CVS totally eradicated :)