Skip to content

Conversation

afresh1
Copy link
Contributor

@afresh1 afresh1 commented May 2, 2019

Without waiting on the forked process, we may check the status of the wrong $?.

Fix from millert@openbsd

This happens at least on OpenBSD and I've heard on FreeBSD using mandoc.

http://mandoc.bsd.lv/

$ cd Pod-Perldoc
$ PERL5LIB=lib ./perldoc -oMan Digest
Error from pipe to /usr/bin/mandoc!

 at lib/Pod/Perldoc.pm line 1517.
Falling back to Pod because there was a problem!

 at lib/Pod/Perldoc.pm line 1517.
Error while formatting with Pod::Perldoc::ToMan:
 Can't read-open Pod::Perldoc::ToMan=HASH(0x43a208de958): No such file or directory
Aborting
 at lib/Pod/Perldoc/ToMan.pm line 407.


 at ./perldoc line 6.
Got a 0-length file from /usr/libdata/perl5/Digest.pm via Pod::Perldoc::ToMan!?

 at ./perldoc line 6.

Without waiting on the forked process, we may check the status of the
wrong $?.

from millert@openbsd
@briandfoy
Copy link
Owner

I've just taken over the maintenance of Pod::Perldoc, so I'll look into this. I know it's years old, and it might take me a bit to catch up.

@briandfoy briandfoy added Type: bug an existing feature does not work Status: needs verification issue needs to be verified Priority: high work on this first labels Dec 5, 2023
@briandfoy briandfoy self-assigned this Dec 5, 2023
@briandfoy briandfoy merged commit 9c1c49f into briandfoy:master Dec 7, 2023
@briandfoy briandfoy added Status: accepted the fix is accepted and removed Status: needs verification issue needs to be verified Priority: high work on this first labels Dec 7, 2023
@briandfoy
Copy link
Owner

It turns out that this use of waitpid can hang on Windows, but I fixed that in #52.

briandfoy added a commit that referenced this pull request Dec 8, 2023
* Refactor out some utility routines, test https (#52), and fix waitpid uses from #39 .
@karenetheridge
Copy link

karenetheridge commented Dec 12, 2023

Wow, nice timing. I just updated to macos 14.1.2 and encountered the error described in https://www.mail-archive.com/[email protected]/msg51445.html, which brought me here.

(For anyone else reading, the environment variable changes described there did work for me -- export PERLDOC=-oTerm PERLDOC_PAGER='less -R'.)

@briandfoy briandfoy added Status: released there is a new release with this fix and removed Status: accepted the fix is accepted labels Feb 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: released there is a new release with this fix Type: bug an existing feature does not work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants