Bug 217 - mdoc2man.pl puts wrong name into ssh.1 man page
Summary: mdoc2man.pl puts wrong name into ssh.1 man page
Status: CLOSED FIXED
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: Documentation (show other bugs)
Version: -current
Hardware: All All
: P2 minor
Assignee: OpenSSH Bugzilla mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-04-16 04:35 AEST by Philip Spencer
Modified: 2004-04-14 12:24 AEST (History)
0 users

See Also:


Attachments
Prevent mdoc2man.pl from using wrong name in man pages (311 bytes, patch)
2002-04-16 04:37 AEST, Philip Spencer
no flags Details | Diff
CORRECTION to previous patch (448 bytes, patch)
2002-04-16 08:03 AEST, Philip Spencer
no flags Details | Diff
AARRGGHH!! How can I make so many mistakes in a simple patch? This third one should be correct! (456 bytes, patch)
2002-04-16 08:16 AEST, Philip Spencer
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Spencer 2002-04-16 04:35:09 AEST
On systems where mdoc2man.pl is used, it puts the wrong name ("sftp" instead of
"ssh") into the ssh.1 man page. For example, under GatewayPorts: "By default,
sftp binds local port forwardings .... GatewayPorts can be used to specify that
sftp should bind local port forwardings ..."

The problem is that ssh.1 includes the following lines:

   when specifying configuration options using the
  .Nm ssh ,
  .Nm scp
  and
  .Nm sftp
  .Fl o
  option.

In mdoc2man.pl the line ".Nm sftp" sets $name to be "sftp" so subsequent
unqualified uses of ".Nm" produce "sftp" instead of "ssh".

The attached patch fixes the problem by not allowing $name to be redefined once
set.
Comment 1 Philip Spencer 2002-04-16 04:37:43 AEST
Created attachment 74 [details]
Prevent mdoc2man.pl from using wrong name in man pages
Comment 2 Philip Spencer 2002-04-16 08:03:07 AEST
Created attachment 75 [details]
CORRECTION to previous patch
Comment 3 Philip Spencer 2002-04-16 08:16:17 AEST
Created attachment 76 [details]
AARRGGHH!! How can I make so many mistakes in a simple patch? This third one should be correct!
Comment 4 Damien Miller 2002-04-17 12:33:15 AEST
Thanks - applied
Comment 5 Damien Miller 2004-04-14 12:24:18 AEST
Mass change of RESOLVED bugs to CLOSED