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

[Baruch Siach <baruch@tkos.co.il>]

Hi Vicente,

On Wed, Aug 26, 2015 at 12:29:28PM +0100, Vicente Olivert Riera wrote:
> On 08/26/2015 12:12 PM, Baruch Siach wrote:
> > 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.
> 
> 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.

Just did.

Thanks,
baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -