From mboxrd@z Thu Jan  1 00:00:00 1970
From: =?ISO-8859-1?Q?J=F6rg?= Krause <joerg.krause@embedded.rocks>
Date: Sat, 02 Apr 2016 18:29:18 +0200
Subject: [Buildroot] [PATCH] yajl: fix static linking in a parallel build
In-Reply-To: <20160402171318.653fec21@free-electrons.com>
References: <1459538292-7023-1-git-send-email-joerg.krause@embedded.rocks>
 <20160402171318.653fec21@free-electrons.com>
Message-ID: <1459614558.18262.3.camel@embedded.rocks>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

On Sa, 2016-04-02 at 17:13 +0200, Thomas Petazzoni wrote:
> Hello,
> 
> On Fri,??1 Apr 2016 21:18:12 +0200, J?rg Krause wrote:
> 
> > 
> > + FOREACH (test ${TESTS})
> > +???GET_FILENAME_COMPONENT(testProg ${test} NAME_WE)
> > +???ADD_EXECUTABLE(${testProg} ${test})
> > +-??TARGET_LINK_LIBRARIES(${testProg} yajl m)
> > ++??TARGET_LINK_LIBRARIES(${testProg} yajl_s m)
> I am not sure to understand why this would fix the parallel build
> issue. The parallel build problem comes from the fact that the
> library
> and tests sit in different directories, and that they are built in
> parallel with each other. Both the yajl and yajl_s libraries are
> built
> in the same directory, separate from the tests directory. So I don't
> quite see how changing from using yajl to yajl_s is going to make a
> difference here. But maybe I'm missing something.

The issue is that we patch yajl (#0002) to build only the static
library in a static build. So there is no shared library target for
CMake to use as a dependency.

J?rg