Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 25 Jun 2001 11:49:59 -0700
From:      Dima Dorfman <dima@unixfreak.org>
To:        Dag-Erling Smorgrav <des@ofug.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sbin/md5 md5.c 
Message-ID:  <20010625184959.E50FB3E31@bazooka.unixfreak.org>
In-Reply-To: <xzp3d8ocsne.fsf@flood.ping.uio.no>; from des@ofug.org on "25 Jun 2001 20:37:41 %2B0200"

next in thread | previous in thread | raw e-mail | index | archive | help
Dag-Erling Smorgrav <des@ofug.org> writes:
> Dima Dorfman <dima@unixfreak.org> writes:
> > AFAIK, OpenBSD rewrote this because of the license.  If we're going to
> > make major modifications to this, we might as well rewrite it in terms
> > of openssl(1).  It gives slightly different output, e.g.:
> > 
> > dima@hornet% openssl md5 /HORNET
> > MD5(/HORNET)= 96fec985ef627b70295e31640483dd5f
> 
> The only difference is one less space (before the =), and if we can
> live with that (or change openssl to add the missing space) there'd be
> no need to write anything - just 'ln /usr/bin/openssl /usr/bin/md5'
> and likewise for sha1, ripemd160 etc.

Two less spaces: one before and after the parenthesis:

	MD5 (/HORNET) = 96fec985ef627b70295e31640483dd5	<-- /sbin/md5
	MD5(/HORNET)= 96fec985ef627b70295e31640483dd5f	<-- openssl md5

openssl(1) also doesn't support -p and -s, which some people use, but
that can be fixed (either in the OpenSSL repo or in ours).  Having
openssl output spaces like ours may not be such a great idea, however,
since some people or programs may depend on openssl's output the way
they already have it.  Much better to change what we have (primarily
bsd.port.mk, I guess) to deal with the lack of spaces, and have it be
another [minor] incompatibility between 4.x and 5.x.

					Dima Dorfman
					dima@unixfreak.org

P.S.  As an aside, I'd rather that we make links like you suggest
instead of telling people to use openssl(1) now since openssl(1) may
go away (we may replace OpenSSL with SomethingElseSSL in the future),
but this functionality shouldn't.

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010625184959.E50FB3E31>