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.

Sign up for GitHub

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

Jump to bottom

Pod::Html loses link #14413

Closed
p5pRT opened this issue Jan 12, 2015 · 6 comments
Closed

Pod::Html loses link #14413

p5pRT opened this issue Jan 12, 2015 · 6 comments
Assignees
@jkeenan
Labels
ext/Pod-Html issues in the blead-upstream Pod-Html distribution type-library type-utilities

Comments

@p5pRT
Copy link

p5pRT commented Jan 12, 2015

Migrated from rt.perl.org#123593 (status was 'open')

Searchable as RT123593$
@p5pRT
Copy link
Author

p5pRT commented Jan 12, 2015

From @Tux

$ cat rt.pl
#!/usr/bin/perl

1;

=head1 DEMO
X<DEMO>

L<DEMO|/DEMO> without NL

L<DEMO with NL|
/DEMO> and trailing text

=cut
$ podchecker rt.pl
rt.pl pod syntax OK.
$ perl -MV=Pod​::Checker,Pod​::Simple,Pod​::Html
Pod​::Checker
  /pro/lib/perl5/5.20.0/Pod/Checker.pm​: 1.71
Pod​::Simple
  /pro/lib/perl5/5.20.0/Pod/Simple.pm​: 3.28
Pod​::Html
  /pro/lib/perl5/5.20.0/Pod/Html.pm​: 1.21
$ pod2text rt.pl
DEMO
  DEMO without NL

  DEMO with NL and trailing text

$ pod2html rt.pl
Cannot find " " in podpath​: cannot find suitable replacement path, cannot resolve link
<?xml version="1.0" ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http​://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http​://www.w3.org/1999/xhtml">
<head>
<title></title>
<meta http-equiv="content-type" content="text/html; charset=utf-8" />
<link rev="made" href="mailto​:h.m.brand@​xs4all.nl" />
</head>

<body>

<ul id="index">
  <li><a href="#DEMO">DEMO </a></li>
</ul>

<h1 id="DEMO">DEMO </h1>

<p><a href="#DEMO">DEMO</a> without NL</p>

<p><a>DEMO with NL</a> and trailing text</p>

</body>

</html>

@p5pRT
Copy link
Author

p5pRT commented Jan 13, 2015

From @jkeenan

On Mon Jan 12 09​:04​:14 2015, hmbrand wrote​:

$ cat rt.pl
#!/usr/bin/perl

1;

=head1 DEMO
X<DEMO>

L<DEMO|/DEMO> without NL

L<DEMO with NL|
/DEMO> and trailing text

=cut
$ podchecker rt.pl
rt.pl pod syntax OK.
$ perl -MV=Pod​::Checker,Pod​::Simple,Pod​::Html
Pod​::Checker
/pro/lib/perl5/5.20.0/Pod/Checker.pm​: 1.71
Pod​::Simple
/pro/lib/perl5/5.20.0/Pod/Simple.pm​: 3.28
Pod​::Html
/pro/lib/perl5/5.20.0/Pod/Html.pm​: 1.21
$ pod2text rt.pl
DEMO
DEMO without NL

DEMO with NL and trailing text

$ pod2html rt.pl
Cannot find " " in podpath​: cannot find suitable replacement path,
cannot resolve link
<?xml version="1.0" ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http​://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http​://www.w3.org/1999/xhtml">
<head>
<title></title>
<meta http-equiv="content-type" content="text/html; charset=utf-8" />
<link rev="made" href="mailto​:h.m.brand@​xs4all.nl" />
</head>

<body>

<ul id="index">
<li><a href="#DEMO">DEMO </a></li>
</ul>

<h1 id="DEMO">DEMO </h1>

<p><a href="#DEMO">DEMO</a> without NL</p>

<p><a>DEMO with NL</a> and trailing text</p>

</body>

</html>

Additional data point. When the newline occurs *before* the pipe in the final L<> link, there is no error and the link is preserved. See attachments.

Thank you very much.

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Jan 13, 2015

From @jkeenan

123593-newline-before-pipe.pl

@p5pRT
Copy link
Author

p5pRT commented Jan 13, 2015

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

@jkeenan
Copy link
Contributor

jkeenan commented Jan 30, 2021

@Tux, I get no warnings when I run your sample program through the pod2html that ships with perl-5.32.0.

$ pod2html rt.pl 2> ./gh-14413-warnings.txt; ls -l gh-14413-warnings.txt
<?xml version="1.0" ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title></title>
<meta http-equiv="content-type" content="text/html; charset=utf-8" />
<link rev="made" href="mailto:jkeenan@zareason.(none)" />
</head>

<body>



<ul id="index">
  <li><a href="#DEMO">DEMO </a></li>
</ul>

<h1 id="DEMO">DEMO </h1>

<p><a href="#DEMO">DEMO</a> without NL</p>

<p><a>DEMO with NL</a> and trailing text</p>


</body>

</html>


-rw-rw-r-- 1 jkeenan jkeenan 0 Jan 30 17:22 gh-14413-warnings.txt

Is this still a problem for you?

Thank you very much.
Jim Keenan

@jkeenan jkeenan added Closable? We might be able to close this ticket, but we need to check with the reporter ext/Pod-Html issues in the blead-upstream Pod-Html distribution labels Jan 30, 2021
@jkeenan jkeenan self-assigned this Jan 31, 2021
@Tux
Copy link
Contributor

Tux commented Jan 31, 2021

@jkeenan I just checked, no warning, looks like being resolved
Feel free to close

@jkeenan jkeenan removed Closable? We might be able to close this ticket, but we need to check with the reporter Severity Low labels Jan 31, 2021
@jkeenan jkeenan closed this as completed Jan 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jkeenan jkeenan

Labels
ext/Pod-Html issues in the blead-upstream Pod-Html distribution type-library type-utilities
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Tux @jkeenan @p5pRT