Re: [Bug-wget] [bug #46161] make check fails due to missing library in tests/Makefile link command

[Rodgers, Kevin]

Hi Tim,

Hmmm, I can't reproduce it either now.  My recipe was the same as yours, except 
for these additional configure options: --with-ssl=openssl --with-openssl=yes

The distribution doesn't include README.checkout and I don't see it on the ftp 
server, but uname -a reports:
Linux testfe3 2.4.21-9.ELsmp #1 SMP Thu Jan 8 17:08:56 EST 2004 i686 i686 i386 
GNU/Linux

I guess the changes I made to get wget to configure and build with OpenSSL 
fixed the linking issue I had with tests/Makefile.  Your advice about mixing 
header and library versions is appreciated.

Sorry for the noise, thanks for a great tool!

Kevin Rodgers 
Principal Software Engineer
Product Development & Design
Tel: 303-397-2807
IHS: 710-2807




-----Original Message-----
From: Tim Rühsen [mailto:address@hidden 
Sent: Saturday, October 10, 2015 6:40 AM
To: address@hidden
Cc: Rodgers, Kevin; address@hidden; address@hidden
Subject: Re: [bug #46161] make check fails due to missing library in 
tests/Makefile link command

Hi Kevin,

I can't reproduce the wget compilation problem.

Steps I did:
- download http://ftp.gnu.org/gnu/wget/wget-1.16.3.tar.gz
- tar xvfz wget-1.16.3.tar.gz
- cd wget-1.16.3/
- ./configure
- make
- make check

You should really give us all details on how to reproduce your problem with 
tests/Makefile.in. Also we need more information about your OS and the versions 
of software you use (see README.checkout for what is relevant to wget 
compilation).

We won't (and we simply can't) support all combinations of library/header 
versions of openssl. In fact, you should really sync library and header version 
being the same, else you maybe run into obscure problems with self- compiled 
software later (a perfect method to achieve an unstable system).

Regards, Tim

Am Freitag, 9. Oktober 2015, 14:30:26 schrieb Rodgers, Kevin:
> Hi Tim,
> 
> Yes, I realized after I submitted my report that tests/Makefile is 
> generated.  But Makefile.in is included in the source distribution, so 
> that is a better file to patch (1 step upstream of tests/Makefile, but 
> still 1 step downstream of tests/Makefile.am).
 
> I did download and build wget 1.16.3 and found that it has the same problem.
>  In addition, I had to hack openssl.c to deal with the fact that the 
> old server I'm working on has libssl.so.0.9.8 installed but its 
> include files are from 0.9.7a (Open SSL):
 
> *** src/openssl.c.orig  2015-02-10 14:23:49.000000000 -0700
> --- src/openssl.c       2015-10-08 15:41:23.000000000 -0600
> ***************
> *** 198,203 ****
> --- 198,206 ----
>   #if OPENSSL_VERSION_NUMBER >= 0x00907000
>     OPENSSL_load_builtin_modules();
>     ENGINE_load_builtin_engines();
> +   #ifndef CONF_MFLAGS_DEFAULT_SECTION
> +   #define CONF_MFLAGS_DEFAULT_SECTION 0x20
> +   #endif
>     CONF_modules_load_file(NULL, NULL,
>         CONF_MFLAGS_DEFAULT_SECTION|CONF_MFLAGS_IGNORE_MISSING_FILE);
>   #endif
> 
> Thanks,
> 
> Kevin Rodgers
> Principal Software Engineer
> Product Development & Design
> Tel: 303-397-2807
> IHS: 710-2807
> 
> 
> 
> -----Original Message-----
> From: Tim Ruehsen [mailto:address@hidden
> Sent: Friday, October 09, 2015 2:10 AM
> To: Tim Ruehsen; Rodgers, Kevin; address@hidden; address@hidden; 
> address@hidden
 Subject: [bug #46161] make check fails due to missing
> library in tests/Makefile link command Update of bug #46161 (project 
> wget):
> 
>                   Status:                    None => Invalid               
> 
 Assigned to:                    None => rockdaboot Fixed Release:        
>            None => 1.16.3 
>     _______________________________________________________
> 
> Follow-up Comment #1:
> 
> Thanks for having a look.
> 
> But tests/Makefile is an auto-generated file (generated from Makefile.am).
> So it is definitely the wrong place to fix anything.
 
> And please always check the latest source code from git, the problem 
> might have already been fixed.
 
> How to get the sources from git:
> https://savannah.gnu.org/git/?group=wget
> 
> After downloading the sources you should read README.checkout to proceed.
> If you still have problems to compile/link, feel free to open a new issue.
> 
> 
>     _______________________________________________________
> 
> Reply to this item at:
> 
>   <http://savannah.gnu.org/bugs/?46161>
> 
> _______________________________________________
>   Message sent via/by Savannah
>   http://savannah.gnu.org/
>