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
Support mandoc in perl5db.t #15299
Comments
From andreas@andreasvoegele.comWhen mandoc is installed instead of the traditional man package one of lib/perl5db ................................................... |
From andreas@andreasvoegele.com0001-Fix-test-failure-in-lib-perl5db.t-when-mandoc-is-ins.patchFrom 98ce25c1c4f53aeb5e935dda65d105d58fdb51d7 Mon Sep 17 00:00:00 2001
From: Andreas Voegele <andreas@andreasvoegele.com>
Date: Tue, 26 Apr 2016 09:08:50 +0200
Subject: [PATCH] Fix test failure in lib/perl5db.t when mandoc is installed
---
lib/perl5db.t | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/perl5db.t b/lib/perl5db.t
index 0c4fc42..e216685 100644
--- a/lib/perl5db.t
+++ b/lib/perl5db.t
@@ -2769,7 +2769,7 @@ SKIP:
);
$wrapper->output_like(
- qr/No manual entry for perlrules/,
+ qr/No (?:manual )?entry for perlrules/,
'perldoc command works fine',
);
}
--
2.8.1
|
From @jkeenanOn Wed Apr 27 22:11:35 2016, voegelas wrote:
You say "instead of". Since I have 'man' installed, if I were to install 'mandoc' how would I be able to trick the test into thinking that 'man' was not installed? -- |
The RT System itself - Status changed from 'new' to 'open' |
From andreas@andreasvoegele.comJames E Keenan writes:
The test in perl5db.t uses /usr/bin/man. /usr/bin/man is either provided by the traditional man package or by the mandoc package. The traditional man program outputs "No manual entry for perlrules". The man program from mandoc outputs "No entry for perlrules in the manual". There may be Linux distributions that provide a mandoc package that does not replace /usr/bin/man and that can be installed in parallel to the man package. But there are mandoc packages that fully replace the traditional man package. It's similar to the vi command, which may be provided by vim, elvis, nvi etc. on Linux. |
From @jkeenanOn Sun May 01 21:44:17 2016, voegelas wrote:
This seems to be a real edge case. The test in question only runs on Linux: ##### But when I go to https://en.wikipedia.org/wiki/Mandoc, I read: ##### When I got to mandoc's own website and look for information as to which OSes it runs on (http://mdocml.bsd.lv/ports.html), I find that mandoc is indeed primarily a BSD program, but has an official port on Arch Linux and unofficial ports to Slackware Linux and Crux Linux. Notably absent: Debian and distributions built on top of Debian, such as Ubuntu (which I have). I downloaded the mandoc source code and acked for the error message cited by OP Andreas Voegele. I found: ##### Since I've found the error message, I'm willing to apply the patch submitted -- but I would like either a +1 from another committer or a confirmation from someone using, say, Arch Linux. Thank you very much. -- |
From @khwilliamsonOn 05/05/2016 05:43 PM, James E Keenan via RT wrote:
+1 from me. There is no harm in that patch, and since it helps pass a
|
From @jkeenanOn Thu May 05 20:24:57 2016, public@khwilliamson.com wrote:
Thanks. Applied to blead in commit 8fe07b3. -- |
@jkeenan - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#128012 (status was 'resolved')
Searchable as RT128012$
The text was updated successfully, but these errors were encountered: