Skip Menu |

This queue is for tickets about the GraphViz2 CPAN distribution.

Report information
The Basics
Id: 69076
Status: resolved
Priority: 0/
Queue: GraphViz2

People
Owner: Nobody in particular
Requestors:
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.03
Fixed in: (no value)



Subject: FAIL GraphViz2-1.03 , no t\html, use lib interference for scripts/parse.isa.pl
there is no t\html directory, so all scripts fail like Can't open(> t\html\Heawood.svg): No such file or directory at lib/GraphViz2.pm line 551. after fixing that, scripts/parse.isa.pl would die with syntax error at lib/GraphViz2.pm line 19, near "fieldhash my " diff -ru GraphViz2-1.03/scripts/parse.isa.pl GraphViz2-1.03-new/scripts/parse.isa.pl --- GraphViz2-1.03/scripts/parse.isa.pl 2011-06-18 23:30:10.000000000 -0700 +++ GraphViz2-1.03-new/scripts/parse.isa.pl 2011-06-24 01:46:41.031250000 -0700 @@ -3,7 +3,6 @@ # Note: t/test.t searches for the next line. # Annotation: Demonstrates graphing a Perl class hierarchy. -use lib 't/lib'; use strict; use warnings; @@ -39,7 +38,8 @@ my($parser) = GraphViz2::Parse::ISA -> new(graph => $graph); # These classes live in t/lib/. - +# use lib 't/lib'; +require lib; lib->import('t/lib'); $parser -> create(class => 'Parent::Child::Grandchild', ignore => []); my($format) = shift || 'svg'; diff -ru GraphViz2-1.03/t/test.t GraphViz2-1.03-new/t/test.t --- GraphViz2-1.03/t/test.t 2011-06-18 23:30:10.000000000 -0700 +++ GraphViz2-1.03-new/t/test.t 2011-06-24 01:55:28.671875000 -0700 @@ -15,16 +15,21 @@ my($count) = 1; # Counting the use_ok above. my(%script) = GraphViz2::Utils -> new -> get_scripts; -my($output) = File::Spec -> catfile('t', 'html', '*'); +my $t_html = File::Spec -> catfile('t', 'html'); +my($output) = File::Spec -> catfile( $t_html, '*'); +mkdir $t_html; unlink glob($output); my($stdout, $stderr); for my $script (sort keys %script) { + next if $script !~ /.pl$/; # gVim + my @args = ( $^X, '-Ilib', $script, 'svg', File::Spec -> catfile($t_html, "$script{$script}.svg") ); +# diag " @args "; $count++; - ($stdout, $stderr) = capture{system $^X, '-Ilib', $script, 'svg', File::Spec -> catfile('t', 'html', "$script{$script}.svg")}; + ($stdout, $stderr) = capture{system @args}; ok(length($stderr) == 0, "$script runs without error"); }
I had this same problem. t/lib/Parent.pm conflicts with the "parent" package (parent − Establish an ISA relationship with base classes at compile time). Removing t/lib/Parent.pm fixes things. Perhaps it would be best to move the classes in t/lib into a separate namespace?
Subject: Re: [rt.cpan.org #69076] FAIL GraphViz2-1.03 , no t\html, use lib interference for scripts/parse.isa.pl
Date: Tue, 08 Nov 2011 08:36:26 +1100
To: bug-GraphViz2 [...] rt.cpan.org
From: Ron Savage <ron [...] savage.net.au>
Hi Kevin On Mon, 2011-11-07 at 15:55 -0500, Kevin Galinsky via RT wrote: Show quoted text
> Queue: GraphViz2 > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=69076 > > > I had this same problem. t/lib/Parent.pm conflicts with the "parent" package (parent − Establish > an ISA relationship with base classes at compile time). Removing t/lib/Parent.pm fixes things. > Perhaps it would be best to move the classes in t/lib into a separate namespace?
Thanx for the report. V 1.04 on June 22nd attempted to deal with the problem of writing to the directory t/html/, by using File::Temp instead. That user referred to t \html, so must have been using Windows. Which version are you using? Which OS are you using? Please try the current version from CPAN, V 1.10, and let me know how it goes. It's important to know if that problem has been fixed or not. As for the parent 'v' Parent problem, that too sounds like a Windows problem, but is one I'm prepared to work on, by renaming Parent.pm. Obviously I'd like to resolve both problems, if there are 2, at the same time, so I'm looking forward to your report :-). -- Ron Savage http://savage.net.au/ Ph: 0421 920 622
Hi Folks The MS Windows problem re the directory t\html was fixed in V 1.04. The MS Windows problem (presumably) re Parent.pm has just been fixed in V 1.11 by renaming that class to Adult.pm.