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
installhtml installs private pod #11859
Comments
From @nwc10./installhtml is installing pod that it isn't meant to. $ cd /home/nick/Sandpit/snap-v5.15.6-450-g6ca15f9/lib/perl5/5.15.6/html None of these should be installed, given that installperl installs none of Probably installhtml should be refactored to use the same exclusion lists I don't know if this is a regression from perl 5.14.0 Nicholas Clark |
From @nwc10On Tue Jan 10 11:28:11 2012, nicholas wrote:
and TestInit.html, from TestInit.pm Likely this is all a side effect of it scanning from the top of the Nicholas Clark |
@nwc10 - Status changed from 'new' to 'open' |
Back in 2012, @nwc10 was reporting that the following files should not have been installed:
and
Attempting to understand this problem (I don't customarily use
I got 858 .html files installed under /tmp/gh-11859-html/lib/perl5/5.33.7. But I didn't get any among those Nick listed. Have I invoked Thank you very much. |
This will facilitate debugging of tickets like Perl#19052 and Perl#11859.
Previously (as reported nine years ago by Nick Clark in what is now Perl#11859), the default setting for 'podpath' was '.'. This meant that, unless you explicitly set a colon-delimited list of directories as the value for 'podpath', all files containing POD under '.' were HTML-ified and installed. The only directories that always should have HTML-ized POD installed are 'lib' and 'pod' itself. Invoking 'installhtml' like the following should now DTRT: export HTMLDIR=$HOMEDIR/tmp/installhtml && \ ./perl -Ilib ./installhtml \ --htmldir=$HTMLDIR \ --htmlroot=$HTMLDIR \ --splithead=pod/perlipc \ --recurse \ --verbose 2>&1 | tee installhtml.log
Should be addressed by #19075. |
CORRECTION: There's a pull request needing review for this issue. |
Previously (as reported nine years ago by Nick Clark in what is now #11859), the default setting for 'podpath' was '.'. This meant that, unless you explicitly set a colon-delimited list of directories as the value for 'podpath', all files containing POD under '.' were HTML-ified and installed. The only directories that always should have HTML-ized POD installed are 'lib' and 'pod' itself. Invoking 'installhtml' like the following should now DTRT: export HTMLDIR=$HOMEDIR/tmp/installhtml && \ ./perl -Ilib ./installhtml \ --htmldir=$HTMLDIR \ --htmlroot=$HTMLDIR \ --splithead=pod/perlipc \ --recurse \ --verbose 2>&1 | tee installhtml.log
The code for #19075 has been merged into blead. That should resolve this ticket. Thank you very much. |
This will facilitate debugging of tickets like Perl/perl5#19052 and Perl/perl5#11859.
Previously (as reported nine years ago by Nick Clark in what is now Perl/perl5#11859), the default setting for 'podpath' was '.'. This meant that, unless you explicitly set a colon-delimited list of directories as the value for 'podpath', all files containing POD under '.' were HTML-ified and installed. The only directories that always should have HTML-ized POD installed are 'lib' and 'pod' itself. Invoking 'installhtml' like the following should now DTRT: export HTMLDIR=$HOMEDIR/tmp/installhtml && \ ./perl -Ilib ./installhtml \ --htmldir=$HTMLDIR \ --htmlroot=$HTMLDIR \ --splithead=pod/perlipc \ --recurse \ --verbose 2>&1 | tee installhtml.log
Migrated from rt.perl.org#107880 (status was 'open')
Searchable as RT107880$
The text was updated successfully, but these errors were encountered: