From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ryan Coe Date: Mon, 20 Apr 2015 19:03:10 -0700 Subject: [Buildroot] [PATCH 1/1] heimdal: needs host-flex and explicitly set -lpthread In-Reply-To: <20150420223532.5677fb5f@free-electrons.com> References: <1429466003-4595-1-git-send-email-bluemrp9@gmail.com> <20150420223532.5677fb5f@free-electrons.com> Message-ID: <5535AFDE.9060704@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On 04/20/2015 01:35 PM, Thomas Petazzoni wrote: > Dear Ryan Coe, > > On Sun, 19 Apr 2015 10:53:23 -0700, Ryan Coe wrote: >> Signed-off-by: Ryan Coe >> --- >> package/heimdal/heimdal.mk | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/package/heimdal/heimdal.mk b/package/heimdal/heimdal.mk >> index 972f15c..c4a1703 100644 >> --- a/package/heimdal/heimdal.mk >> +++ b/package/heimdal/heimdal.mk >> @@ -6,11 +6,11 @@ >> >> HEIMDAL_VERSION = 1.5.3 >> HEIMDAL_SITE = http://www.h5l.org/dist/src >> -HEIMDAL_DEPENDENCIES = host-e2fsprogs host-pkgconf >> +HEIMDAL_DEPENDENCIES = host-e2fsprogs host-pkgconf host-flex >> HEIMDAL_INSTALL_STAGING = YES >> # static because of -fPIC issues with e2fsprogs on x86_64 host >> HOST_HEIMDAL_CONF_OPTS = --with-x=no --disable-shared --enable-static >> -HOST_HEIMDAL_CONF_ENV = MAKEINFO=true >> +HOST_HEIMDAL_CONF_ENV = MAKEINFO=true LIBS="-lpthread" > Could you give more details about the situations in which you are > seeing problems with host-flex and -lpthread being missing? > > I indeed see that flex and bison are being tested at configure time, > but failing to detect them does not seem to affect the build (maybe > heimdal has a pre-generated versions of the files normally generated by > flex/bison?). And I see -lpthread being used properly when linking. > > So, please include more details about the issues you're seeing your > commit log. > > Thanks a lot! > > Thomas Thomas, I found the addition of host-flex not to be necessary. The -lpthread addition fixes a build error on my machine. I will submit a new version of my patch. Thanks, Ryan