Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pod::To::Text failing on uneven row lengths #5746

Closed
p6rt opened this issue Oct 12, 2016 · 9 comments
Closed

Pod::To::Text failing on uneven row lengths #5746

p6rt opened this issue Oct 12, 2016 · 9 comments

Comments

@p6rt
Copy link

p6rt commented Oct 12, 2016

Migrated from rt.perl.org#129862 (status was 'resolved')

Searchable as RT129862$

@p6rt
Copy link
Author

p6rt commented Oct 12, 2016

From @dwarring

Consider pod.pl​:

=begin pod

=begin table
a | b | c
l | m | n
x | y
=end table

=end pod

% perl6 --doc /tmp/pod.pl

Method 'chars' not found for invocant of class 'Any'
  in block at /home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 71
  in sub table2text at
/home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 70
  in sub pod2text at
/home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 23
  in block at /home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 27
  in sub pod2text at
/home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 18
  in sub named2text at
/home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 52
  in sub pod2text at
/home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 21
  in block at /home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 27
  in sub pod2text at
/home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 18
  in method render at
/home/david/git/rakudo-stable/install/share/perl6/sources/5DD1D8B49C838828E13504545C427D3D157E56EC
(Pod​::To​::Text) line 3

@p6rt
Copy link
Author

p6rt commented Oct 12, 2016

From @tbrowder

On Wed, Oct 12, 2016 at 3​:22 PM, David Warring <perl6-bugs-followup@​perl.org>
wrote​:

# New Ticket Created by David Warring
# Please include the string​: [perl #​129862]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=129862 >

Consider pod.pl​:

=begin pod

=begin table
a | b | c
l | m | n
x | y
=end table

With the current state of rakudo
​ pod​
, ill-formed tables are problematic. Try redoing the table like this
(adding the separator for the missing column, making the column spacing the
same, lining up the column separators)​:

=begin table
a | b | c
l | m | n
x | y |
​ ​
=end table

​It still might not work with the empty cell.

@p6rt
Copy link
Author

p6rt commented Oct 12, 2016

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Oct 12, 2016

From @dwarring

I got this error from a typo in a large table. Needed to do a little work to find the problematic row.

Have raised Rakudo PR#​903 which hopefully addressees this

On Wed Oct 12 13​:34​:45 2016, tbrowder wrote​:

On Wed, Oct 12, 2016 at 3​:22 PM, David Warring <perl6-bugs-
followup@​perl.org>
wrote​:

# New Ticket Created by David Warring
# Please include the string​: [perl #​129862]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=129862 >

Consider pod.pl​:

=begin pod

=begin table
a | b | c
l | m | n
x | y
=end table

With the current state of rakudo
​ pod
, ill-formed tables are problematic. Try redoing the table like this
(adding the separator for the missing column, making the column
spacing the
same, lining up the column separators)​:

=begin table
a | b | c
l | m | n
x | y |
=end table

​It still might not work with the empty cell.

@p6rt
Copy link
Author

p6rt commented Oct 14, 2016

From @zoffixznet

PR merged[^1]; tests needed.

[1] rakudo/rakudo@7af9ec9

On Wed Oct 12 13​:50​:31 2016, david.warring wrote​:

I got this error from a typo in a large table. Needed to do a little
work to find the problematic row.

Have raised Rakudo PR#​903 which hopefully addressees this

On Wed Oct 12 13​:34​:45 2016, tbrowder wrote​:

On Wed, Oct 12, 2016 at 3​:22 PM, David Warring <perl6-bugs-
followup@​perl.org>
wrote​:

# New Ticket Created by David Warring
# Please include the string​: [perl #​129862]
# in the subject line of all future correspondence about this
issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=129862 >

Consider pod.pl​:

=begin pod

=begin table
a | b | c
l | m | n
x | y
=end table

With the current state of rakudo
​ pod
, ill-formed tables are problematic. Try redoing the table like this
(adding the separator for the missing column, making the column
spacing the
same, lining up the column separators)​:

=begin table
a | b | c
l | m | n
x | y |
=end table

​It still might not work with the empty cell.

@p6rt
Copy link
Author

p6rt commented Oct 14, 2016

From @tbrowder

On Thu, Oct 13, 2016 at 8​:32 PM, Zoffix Znet via RT
<perl6-bugs-followup@​perl.org> wrote​:

PR merged[^1]; tests needed.

[1] rakudo/rakudo@7af9ec9

See perl/roast PR 168.

@p6rt
Copy link
Author

p6rt commented Oct 14, 2016

From @zoffixznet

Tests added in Raku/roast@f1ab1c7

@p6rt
Copy link
Author

p6rt commented Oct 14, 2016

@zoffixznet - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed Oct 14, 2016
@p6rt
Copy link
Author

p6rt commented Nov 30, 2017

From @tbrowder

Root problem is fixed by PR #​1240.

@p6rt p6rt added the testneeded label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant