[Buildroot] [PATCH] package/tn5250: fix static linking problem

[Vicente Olivert Riera <Vincent.Riera@imgtec.com>]

Dear Baruch Siach,

On 08/26/2015 12:12 PM, Baruch Siach wrote:
> Hi Vicente,
> 
> On Wed, Aug 26, 2015 at 12:04:09PM +0100, Vicente Olivert Riera wrote:
>> On 08/26/2015 11:32 AM, Baruch Siach wrote:
>>> On Wed, Aug 26, 2015 at 10:09:31AM +0100, Vicente Olivert Riera wrote:
>>>> On 08/26/2015 10:08 AM, Baruch Siach wrote:
>>>>> On Wed, Aug 26, 2015 at 09:59:51AM +0100, Vicente Olivert Riera wrote:
>>>>>>  ifeq ($(BR2_PACKAGE_OPENSSL),y)
>>>>>> +TN5250_CONF_ENV += LIBS="-lssl -lcrypto -lz"
>>>>>
>>>>> Please use pkg-config. See package/vsftpd/vsftpd.mk for example.
>>>>
>>>> that was my first approach and it didn't work. Does it work for you?
>>>
>>> Please try http://patchwork.ozlabs.org/patch/510797/. Works for me :-).
>>
>> yeah, that one works. I was missing "libcrypto". But, I don't understand
>> why do you need to put libcrypto manually since it's already listed in
>> the openssl.pc file, and then libcrypto pulls libz. Doesn't work
>> recursively?
> 
> I tried "--libs openssl" here, and it works as well. I'm not sure which one is 
> better.
> 
> baruch
> 

indeed, it works. Now I'm wondering which which mistake I made.

I think just "--libs openssl" is better since it seems to act
recursively and pull all the other libraries as well. If you send a v2
of your patch, you can keep my reviewed-by and my tested-by, since I
have already done the test.

Regards,

Vincent.