Takers wanted - a perl job

Leopold Toetsch

unread,

Apr 12, 2005, 5:08:30 AM4/12/05

to Perl 6 Internals

t/src/manifest.t tests 3 and 4 used to compare MANIFEST file entries
against CVS/Entries. The latter is now .svn/entries with an xmlish syntax.
The job is now to replace t/src/manifest.t:scan_cvs so that it extracts
C<name> items from .svn/entries and descends into C<kind="dir">
subdirectories. And no - I don't think, we need a full-fledged XML
parser for this.

Thanks,
leo

Nicholas Clark

unread,

Apr 12, 2005, 5:54:14 AM4/12/05

to Michael G Schwern, Leopold Toetsch, Perl 6 Internals

On Tue, Apr 12, 2005 at 02:50:57AM -0700, Michael G Schwern wrote:

> No need to parse the XML files, "svn list -R" lists everything in the repo.
>
> And I suppose I just volunteered myself for the job.

$ svn list -R
svn: '.' is not a working copy

Doesn't work when svk is used to check out the copy. But in that case
svk list -R does.

Nicholas Clark

Michael G Schwern

unread,

Apr 12, 2005, 5:50:57 AM4/12/05

to Leopold Toetsch, Perl 6 Internals

No need to parse the XML files, "svn list -R" lists everything in the repo.

Ron Blaschke

unread,

Apr 12, 2005, 6:09:51 AM4/12/05

to Leopold Toetsch, Perl 6 Internals

No XML parser? Bummer, it would be sooo much simpler. ;-)

C:\parrot\.svn>perl -MXML::Simple -MData::Dumper
-e"print(join \"\n\", keys %{XMLin('entries')->{entry}})"

encodings
ops
pf
ast
PBC_COMPAT
MANIFEST
...
VERSION
build_tools
README

Ron

Michael G Schwern

unread,

Apr 12, 2005, 3:48:35 PM4/12/05

to Leopold Toetsch, Perl 6 Internals

Hmm. Maybe this should be a commit action and not a test.

Robert Spier

unread,

Apr 13, 2005, 11:58:19 AM4/13/05

to Perl 6 Internals

> > Doesn't work when svk is used to check out the copy. But in that case
> > svk list -R does.
>
> Hmm. Maybe this should be a commit action and not a test.

It was under CVS. I'm pretty sure everyone ignored it there :)

-R

Leopold Toetsch

unread,

Apr 13, 2005, 4:31:58 PM4/13/05

to Robert Spier, Perl 6 Internals

Well, it always depends, how responds looks like:

Committed revision 1234

*********************************
ATT MAINFEST ERROR
missing bla.bla ...
*********************************

> -R

leo

Michael G Schwern

unread,

Apr 13, 2005, 4:05:21 PM4/13/05

to Robert Spier, Perl 6 Internals

They wouldn't ignore it if it caused the commit to fail now would they.

Robert Spier

unread,

Apr 17, 2005, 12:52:26 AM4/17/05

to Perl 6 Internals

>
> Well, it always depends, how responds looks like:
>
> Committed revision 1234
>
> *********************************
> ATT MAINFEST ERROR
> missing bla.bla ...
> *********************************

This is very similar to what it did under CVS, although maybe not
quite as big.

Anyway, if this is something the community wants, if someone could
implement a first pass at the hook script, we can run it. It'll need
to use svnlook cat to look at the Manifest, svnlook changed to detect
new files or removed files, and then compare. Actual details are up
to the implementer.

Many thanks to Garrett Goebel who created our CVS implementation.

-R

Robert Spier

unread,

May 2, 2005, 1:01:09 AM5/2/05

to Leopold Toetsch, Perl 6 Internals

That is a little harder to miss. :)

In CVS, it was just..

manicheck: Not in MANIFEST: $file
manicheck: No such file: $file

(Anyway, as said before - if someone writes the script - we'll run
it.)

-R