cvs commit: src/usr.bin/whois whois.c

[Hiten Pandya]

David Rhodus wrote:
On Thu, 17 Feb 2005 13:19:36 +0100, Devon H. O'Dell
<dodell at xxxxxxxxxxxxxxx> wrote:
On Thu, 2005-02-17 at 03:53 -0800, Liam J. Foy wrote:

liamfoy     2005/02/17 03:53:33 PST

DragonFly src repository

 Modified files:
   usr.bin/whois        whois.c
 Log:
 - Better handling of a given(-p) invalid port number.
 From: OpenBSD
Don't mean to be picky, but should we start citing revision numbers when
we pick something out of another tree?
--Devon

Whoops, forgot to send this to the list. I notice that lately revisions
aren't being cited, and it is helpful when you want to know where
they're from.
It'd be cool if the revisions could also include a link to the CVS cross
reference eventually, where we could give a "hint" that a revision comes
from another tree.
--Devon




I'm not sure I like that idea.  I think the commit emails are already
too messy with the http:// links to all the diffs.  Personally when
ever I look at pervious revision history I use cvs on the command
line.  How many people use the http:// links in these commit emails ? 
Perhaps we could remove them.

I know I do; because I miss Perforce's ability to stick a small diff of 
the changes into the change log -- this is why we originally started 
using URLs.  They are quite useful, atleast to me and people reviewing 
changes can just click URLs instead of going through the rigmarole of 
doing command-line based diffs.

I would suggest that we retain the http links.

     Hiten