From: Gustavo Zacarias <gustavo@zacarias.com.ar>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 3/3] libpcap: canusb support needs pthreads
Date: Tue, 26 Jun 2012 11:43:12 -0300 [thread overview]
Message-ID: <4FE9CA80.7050203@zacarias.com.ar> (raw)
In-Reply-To: <20120626162321.0859b162@skate>
On 06/26/12 11:23, Thomas Petazzoni wrote:
> Le Tue, 26 Jun 2012 11:19:50 -0300,
> Gustavo Zacarias <gustavo@zacarias.com.ar> a ?crit :
>
>> canusb support in libpcap uses pthreads but the configure script doesn't
>> account for -lpthread usage so it doesn't get added into LIBS which in
>> turn doesn't add the appropiate entry in pcap-config, causing build
>> failures for programs that link to libpcap that don't use pthreads
>> themselves (for example tcpdump).
>> Also add the conditional dependency on BR2_PACKAGE_LIBUSB for canusb.
>
> So maybe we need an additional BR2_TOOLCHAIN_HAS_THREADS dependency
> somewhere?
>
> Thomas
canusb support is autodetected, which basically looks for linux and
libusb-1.
In fact there's another patch from someone else that enables (sic) the
disable option which isn't functional for the current release of
libpcap, so any tweak in the package won't work without it.
And IIRC libusb-1 needs threads, and if there's no libusb there's no
canusb which is the only bit that uses threads, so i think libusb should
probably be fixed rather than libpcap regarding that.
I'll give it a shot later to confirm or deny this and make a proper patch.
In any case it would be something else appart from what this patch fixes
(a big build failure with newish binutils).
Regards.
next prev parent reply other threads:[~2012-06-26 14:43 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-26 14:19 [Buildroot] [PATCH 1/3] samba: bump to version 3.6.6 Gustavo Zacarias
2012-06-26 14:19 ` [Buildroot] [PATCH 2/3] busybox 1.20.1: add patches for mke2fs and ps Gustavo Zacarias
2012-06-26 14:19 ` [Buildroot] [PATCH 3/3] libpcap: canusb support needs pthreads Gustavo Zacarias
2012-06-26 14:23 ` Thomas Petazzoni
2012-06-26 14:43 ` Gustavo Zacarias [this message]
2012-06-26 21:19 ` [Buildroot] [PATCH 1/3] samba: bump to version 3.6.6 Peter Korsgaard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FE9CA80.7050203@zacarias.com.ar \
--to=gustavo@zacarias.com.ar \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox