[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] Current wget --version is slightly screwy

From: Giuseppe Scrivano
Subject: Re: [Bug-wget] Current wget --version is slightly screwy
Date: Mon, 08 Nov 2010 01:45:07 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux)

Hello Micah,

we can assume md5 is always present, this is what the gnulib

I am not sure if somebody is using this information so it is better to
use a constant "+md5" or if we have to drop it completely from the build
info output.  I am a bit tempted by the later solution.

Is somebody (who follows this mailing list) using this information?


Micah Cowan <address@hidden> writes:

> When I build current wget trunk, --version has:
> GNU Wget 1.12-2441 built on linux-gnu.
> +digest +https +ipv6 +iri +large-file -md5 +nls -ntlm +opie +ssl/gnutls
> ...
> Note the -md5, which obviously has to be a lie.
> I guess this is because build_info.c.in has:
> md5 choice:
>     builtin     defined HAVE_BUILTIN_MD5
>     openssl     defined HAVE_OPENSSL_MD5
>     solaris     defined HAVE_SOLARIS_MD5
> but we don't set any of those in configure.ac anymore?
> (According to my config.log, it's coming from gnulib, which is what
> would've corresponded to "builtin")
> I'm guessing we build the md5 component from gnulib unconditionally now,
> because it was too much of a pain to maintain that portion of gnulib as
> a separate fallback bit.
> It's probably a bad idea in general to remove any of the build-info
> strings from the --version output, in case someone is using it
> programatically (which is what +https was intended for); but this
> particular build-info tag was intended more for debugging (i.e., if
> there's something broken with MD5 sums, whose implementation are we
> using)? So we're probably safe just to ditch it; the other option would
> be to hard-code it to "builtin" (if that matches actual fact).

reply via email to

[Prev in Thread] Current Thread [Next in Thread]