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 generates possibly bogus warnings #11860
Comments
From @nwc10installhtml now generates many seemingly troublesome warnings. For example, Cannot find "IPC::Open3" in podpath: more than one possible replacement path to IPC::Open3, using lib::IPC::Open3 It's not clear whether this is actually a genuine problem, as it doesn't look (installhtml on perl 5.14.0 generates 841 lines on STDERR, which is bad, This may be a fundamental problem of how installhtml is currently written 1: find the list of Pod files that need to be installed It might be that duplicate files in lib/ and in {ext,dist,cpan} are actually This might also solve bug #107880 as a side effect. Nicholas Clark |
From ponguile@gmail.comOn Tue, Jan 10, 2012 at 2:56 PM, Nicholas Clark
These warnings are issued by Pod::Html. They are a result of Pod::Html A rather simple fix is to only display these warnings if --verbose is
|
The RT System itself - Status changed from 'new' to 'open' |
From @nwc10On Thu, Jan 12, 2012 at 10:08:30PM -0500, Marc Green wrote:
Aha right. Thanks.
I think that that actually might just be brushing the real problem under --podroot=. --podpath=. --recurse I suspect that that should be 'lib' instead of '.'. Or maybe 'pod:lib:utils' Although, really, I'm still suspecting that *that* is only a medium-term Nicholas Clark |
From @nwc10On Thu, Jan 12, 2012 at 10:08:30PM -0500, Marc Green wrote:
Also, I find the text of the error message a bit troubling. I keep thinking [Some of the manpage scanning code etc *has* had bugs where it's done things If it's about a filename, is it possible to report the filename instead? Nicholas Clark |
From ponguile@gmail.com
I agree that --podpath should be limited to the directories that have POD On Fri, Jan 13, 2012 at 5:18 AM, Nicholas Clark <nick@ccl4.org> wrote:
You think Pod::Html linked to itself instead of the module it is I think the warning message is clear, but I am undoubtedly biased, as I
Is the problem the arisdottle, or the fact that 'lib/' is prefixed? I am
It's reporting about the path to the file. |
From @nwc10On Fri, Jan 13, 2012 at 11:33:32AM -0500, Marc Green wrote:
<off topic>
as below:
Then why is it using "::" in the message rather than "/"? That's my concern. If I see "::" in a message, I assume that we're talking All this confusion and potential for bugs being because things like this: foo/bar/baz.pm Is that bar::baz as found in ./foo? If output starts reporting "You said bar::baz, but I'm using foo::bar::baz" Whereas if it said "You said bar::baz, I'm using bar::baz I found in foo/" Nicholas Clark |
From ponguile@gmail.com
I remember stumbling across a Paamayim Nekudotayim error in PHP a few years
Simply because the destination of some L<> formatting codes contain "::" in
I am in agreement with you. I think the warning message should be clarified Also, I still think that the warning message should be issued only `if |
From @nwc10On Tue, Jan 17, 2012 at 08:40:59PM -0500, Marc Green wrote:
That's somewhat my point. Maybe I'm too subtle, but I didn't stick a smiley
Does this mean that if I have Pod with a link that is L<Open3> it will
If the answer to my above question is yes, I suspect that it (at best) Nicholas Clark |
From @cpansproutOn Wed Jan 18 05:43:17 2012, nicholas wrote:
What’s wrong with multilingual (or polylingual/multiglottal) projects? From locale.t: /* Some day I’ll slip ἐνθάδε πάρεισι δράκοντες into the source. :-) -- Father Chrysostomos |
From ponguile@gmail.com
From the code: ... if there is no exact match then ... It will check for an exact match, and since one does not exist, it will
That is a good point -- in this case it may be better to be safe than |
From ponguile@gmail.comAlthough my commit does not fix the real issue at hand, I have provided marcgreen/perl@a83bce1 that needs to be smoked. However, the work in this patch was done after the work in my other patch, marcgreen/perl@2fdb0c8 -- fixing pod2html's cross referencing, and I checkout out of that branch instead of blead, so this shouldn't be smoked until that one is merged. (Unless someone feels like cherry-picking, then I think it work be OK.) As discussed, this patch only sweeps the problem under the rug. I feel that |
It appears that marcgreen/perl@2fdb0c8 was eventually pulled into blead as 707a94f I can't find any evidence that marcgreen/perl@a83bce1 was ever |
Migrated from rt.perl.org#107882 (status was 'open')
Searchable as RT107882$
The text was updated successfully, but these errors were encountered: