[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: gnu-web-doc-update: fails to add new directories
From: |
Akim Demaille |
Subject: |
Re: gnu-web-doc-update: fails to add new directories |
Date: |
Sun, 24 Jan 2016 19:03:08 +0100 |
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 :)