Beefy Boxes and Bandwidth Generously Provided by pair Networks
Just another Perl shrine

Maybe a bug in Apache::CVS::Directory

by shushu (Scribe)
on Nov 07, 2002 at 09:50 UTC ( #211028=note: print w/replies, xml ) Need Help??

in reply to Re: Re: Re: CVS repository via Perl
in thread CVS repository via Perl

Below is the mail I sent to John Barbee:

Hi John,

When working I am trying to get a directory object and go over all of its sub directories.
I found out that when I am asking for the name of the subdir object, I get an error message.

The error message is:
Can't call method "extension" on an undefined value at /usr/lib/perl5/site_perl/5.6.1/Apache/CVS/ line 231 (#1 +) (F) You used the syntax of a method call, but the slot filled by t +he object reference or package name contains an undefined value. Som +ething like this will reproduce the error:
Looking on the subdir object I saw its rcsconfig object is undef.
Looking in sub load in Apache::CVS::Directory I saw the following:
foreach my $path ( @directories ) { $self->push('directory', Apache::CVS::Directory->new($self->path() . "/$pat +h")); }
foreach my $path ( @files ) { if ( $path =~ /,v$/ ) { $self->push('file', Apache::CVS::File->new($self->path() . "/$path +", $self->rcs_confi +g())); } else { $self->push('plain_file', Apache::CVS::PlainFile->new($self->path() . "/ +$path")); } }

The bottom line is - just by fixing the directory object creation to work like the file creation by sending the $self->rcsconfig object it seems I fixed the problem.
Is it a bug ?
Maybe you did this for a reason, and I harm something else by fixing it like that ?
If it is a bug, and this is a good fix, how do we proceed to release a new version ?
And I will appriciate a quick response,

Log In?

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://211028]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others contemplating the Monastery: (1)
As of 2021-12-04 00:44 GMT
Find Nodes?
    Voting Booth?
    R or B?

    Results (30 votes). Check out past polls.