Bug #86432 for GraphViz2: scripts/record.{1,2}.pl fail in testing

Wed Jun 26 06:28:14 2013 KENTNL [...] cpan.org - Ticket created

Subject:

scripts/record.{1,2}.pl fail in testing

perl -Ilib scripts/record.1.pl

https://gist.github.com/kentfredric/5866383/raw/f5dd5a32b22e25350ffb6e331a70433cef20f049/record.1.pl.out

perl -Ilib scripts/record.2.pl

https://gist.github.com/kentfredric/5866383/raw/712f799f8a39e9ca21ca600861d60fb31a004e68/record.2.pl.out

It would appear its not liking the literal \n in the label.

Manually copying the generated "dot" output into a text file, correcting the \n => solves this problem.

Oh, btw, Fredric . Everyone spells it wrong, and everyone spells it wrong differently. And to add insult to injury, one of my great grandfathers had the same quirk, so as a result the spelling changed somewhere in our ancestry on paper :/

Wed Jun 26 19:21:33 2013 ron [...] savage.net.au - Correspondence added

Subject:	Re: [rt.cpan.org #86432] scripts/record.{1,2}.pl fail in testing
Date:	Thu, 27 Jun 2013 09:21:11 +1000
To:	bug-GraphViz2 [...] rt.cpan.org
From:	Ron Savage <ron [...] savage.net.au>

Hi Kent Thanx for the quick checks. More below. On 26/06/13 20:28, Kent Fredric via RT wrote: Show quoted text

> Wed Jun 26 06:28:14 2013: Request 86432 was acted upon. > Transaction: Ticket created by KENTNL > Queue: GraphViz2 > Subject: scripts/record.{1,2}.pl fail in testing > Broken in: 2.10 > Severity: (no value) > Owner: Nobody > Requestors: KENTNL@cpan.org > Status: new > Ticket<URL: https://rt.cpan.org/Ticket/Display.html?id=86432> > > > perl -Ilib scripts/record.1.pl > > https://gist.github.com/kentfredric/5866383/raw/f5dd5a32b22e25350ffb6e331a70433cef20f049/record.1.pl.out > > perl -Ilib scripts/record.2.pl > > https://gist.github.com/kentfredric/5866383/raw/712f799f8a39e9ca21ca600861d60fb31a004e68/record.2.pl.out > > It would appear its not liking the literal \n in the label.

Correct. On my machine (Debian) dot accepts the text on 2 lines and treats is as a literal '\n'. That's why my tests worked. I just did another test to confirm. Nevertheless, the code is wrong in both programs, since I should have used \\n to get a literal \n in the output. V 2.11 on it's way in 10 minutes. Show quoted text

> Manually copying the generated "dot" output into a text file, correcting the \n > => solves this problem. > > Oh, btw, Fredric . Everyone spells it wrong, and everyone spells it wrong > differently. And to add insult to injury, one of my great grandfathers had the > same quirk, so as a result the spelling changed somewhere in our ancestry on > paper :/

My apologies. That's definitely my mistake. I tried to keep check on the trailing lack-of-k and got that wrong, and also left out the 2nd r. The Changes file has been edited. I did the same thing many years ago with the Perl programmer Kip Hampton, calling him Kim. Understandable (perhaps unfortunately) but wrong. -- Ron Savage http://savage.net.au/ Ph: 0421 920 622

Wed Jun 26 19:21:34 2013 The RT System itself - Status changed from 'new' to 'open'

Thu Jun 27 09:39:02 2013 KENTNL [...] cpan.org - Broken in 2.11 added

Thu Jun 27 09:39:02 2013 KENTNL [...] cpan.org - Broken in 2.12 added

Thu Jun 27 09:39:03 2013 KENTNL [...] cpan.org - Correspondence added

Unfortunately, it seems the fixes added in 2.11 only fixed test #2, not #1

"struct3" [ label="hello
world |{ b |{c|<here> d|e}| f}| g | h" ]

...

Warning: <stdin>:8: string ran past end of line
Error: <stdin>:9: syntax error near line 9
context: world >>> | <<< { b |{c|<here> d|e}| f}| g | h" ]
Warning: <stdin>:9: string ran past end of line
Warning: node struct3, port here unrecognized

Thanks =)

Thu Jun 27 19:59:55 2013 ron [...] savage.net.au - Correspondence added

Subject:	Re: [rt.cpan.org #86432] scripts/record.{1,2}.pl fail in testing
Date:	Fri, 28 Jun 2013 09:59:28 +1000
To:	bug-GraphViz2 [...] rt.cpan.org
From:	Ron Savage <ron [...] savage.net.au>

Hi Kent And what about V 2.12? On 27/06/13 23:39, Kent Fredric via RT wrote: Show quoted text

> Queue: GraphViz2 > Ticket<URL: https://rt.cpan.org/Ticket/Display.html?id=86432> > > Unfortunately, it seems the fixes added in 2.11 only fixed test #2, not #1 > > "struct3" [ label="hello > world |{ b |{c|<here> d|e}| f}| g | h" ] > > ... > > Warning:<stdin>:8: string ran past end of line > Error:<stdin>:9: syntax error near line 9 > context: world>>> |<<< { b |{c|<here> d|e}| f}| g | h" ] > Warning:<stdin>:9: string ran past end of line > Warning: node struct3, port here unrecognized > > > Thanks =) >

-- Ron Savage http://savage.net.au/ Ph: 0421 920 622

Thu Jun 27 21:50:05 2013 KENTNL [...] cpan.org - Correspondence added

On 2013-06-28 11:59:55, ron@savage.net.au wrote:
Show quoted text

> Hi Kent
>
> And what about V 2.12?
>

V2.12 fails the same way v2.11 does.

Thu Jun 27 22:07:24 2013 ron [...] savage.net.au - Correspondence added

Subject:	Re: [rt.cpan.org #86432] scripts/record.{1,2}.pl fail in testing
Date:	Fri, 28 Jun 2013 12:07:01 +1000
To:	bug-GraphViz2 [...] rt.cpan.org
From:	Ron Savage <ron [...] savage.net.au>

Hi Kent On 28/06/13 11:50, Kent Fredric via RT wrote: Show quoted text

> Queue: GraphViz2 > Ticket<URL: https://rt.cpan.org/Ticket/Display.html?id=86432> > > On 2013-06-28 11:59:55, ron@savage.net.au wrote:

>> Hi Kent >> >> And what about V 2.12? >>

> > V2.12 fails the same way v2.11 does.

Correct. I only thought I'd patched both record.1.pl and record.2.pl. In fact the former was un-patched. My apologies. That was careless. V 2.13 has been uploaded to CPAN. -- Ron Savage http://savage.net.au/ Ph: 0421 920 622

Thu Jun 27 23:03:32 2013 KENTNL [...] cpan.org - Correspondence added

Cheers. Fixed in 2.13 =)

Thu Jun 27 23:10:45 2013 ron [...] savage.net.au - Correspondence added

Subject:	Re: [rt.cpan.org #86432] scripts/record.{1,2}.pl fail in testing
Date:	Fri, 28 Jun 2013 13:10:20 +1000
To:	bug-GraphViz2 [...] rt.cpan.org
From:	Ron Savage <ron [...] savage.net.au>

Hi Kent On 28/06/13 13:03, Kent Fredric via RT wrote: Show quoted text

> Queue: GraphViz2 > Ticket<URL: https://rt.cpan.org/Ticket/Display.html?id=86432> > > Cheers. Fixed in 2.13 =)

Fab. And after only about 20 (sic) module releases this month, thanx to your - and CPAN Tester - efforts. I /do/ appreciate it you know :-). This time I really will close this ticket. -- Ron Savage http://savage.net.au/ Ph: 0421 920 622

Thu Jun 27 23:11:29 2013 RSAVAGE [...] cpan.org - Correspondence added

Fixed in V 2.13.

Thu Jun 27 23:11:29 2013 RSAVAGE [...] cpan.org - Status changed from 'open' to 'resolved'