Bug 1728 - unnecessary "./" in slogin and slogin.1 symlinks
Summary: unnecessary "./" in slogin and slogin.1 symlinks
Status: CLOSED WONTFIX
Alias: None
Product: Portable OpenSSH
Classification: Unclassified
Component: Build system (show other bugs)
Version: 5.3p1
Hardware: Other Linux
: P2 trivial
Assignee: Assigned to nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-02 01:50 AEDT by Colin Watson
Modified: 2010-04-16 15:50 AEST (History)
2 users (show)

See Also:


Attachments
simplify slogin and slogin.1 symlinks (697 bytes, patch)
2010-03-02 01:50 AEDT, Colin Watson
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Colin Watson 2010-03-02 01:50:32 AEDT
Created attachment 1802 [details]
simplify slogin and slogin.1 symlinks

I'm almost afraid to file this because it's so trivial :-)

Debian packages are often run through an automated lint-type program called lintian, which checks for a wide variety of problems from the severe to the trivial.  We usually try to keep its output short in order to make it easier to see severe problems.  At the trivial end of the scale, it complains about symlinks with unnecessary segments like "./".  There's certainly no benefit in these, so would you apply the attached patch to simplify the links and shut it up?

Thanks!
Comment 1 Tim Rice 2010-03-02 04:45:05 AEDT
(In reply to comment #0)
> Created an attachment (id=1802) [details]
> simplify slogin and slogin.1 symlinks
[....]
> "./".  There's certainly no benefit in these, so would you apply the
> attached patch to simplify the links and shut it up?

Actually there is a benifit. Without the "./" in the symlink, the packiging tools on a number of SVR4 based platforms will get it wrong.
Comment 2 Colin Watson 2010-03-02 06:45:06 AEDT
Seriously?  Wow, incompatible packaging systems.

I suppose I can keep the patch locally.  I was just trying to reduce my unforwarded-patch count.
Comment 3 Damien Miller 2010-03-26 11:05:35 AEDT
Looks like we get to keep this wart :(
Comment 4 Damien Miller 2010-04-16 15:50:28 AEST
Mass move of bugs RESOLVED->CLOSED following the release of openssh-5.5p1