Can we migrate this thread to https://bugs.linaro.org/show_bug.cgi?id=1787, I think Christoph is looking.
On 3 September 2015 at 08:23, Stuart Haslam <[email protected]> wrote: > On Wed, Sep 02, 2015 at 09:57:34AM +0200, Nicolas Morey-Chaisemartin wrote: > > Hi, > > > > Since all tests were moved to Library I have been hitting the same bug > over and over in the build system. > > Often when I pull master and try to rebuild from a pre existing build, I > have this error: > > libtool: link: rm -fr .libs/libcrypto.a .libs/libcrypto.la > > libtool: link: (cd .libs/libcrypto.lax/libcrypto.a && ar x > "/work1/nmorey/workspace/alternates/master/odp-patches/test/validation/crypto/.libs/libcrypto.a") > > ar: > /work1/nmorey/workspace/alternates/master/odp-patches/test/validation/crypto/.libs/libcrypto.a: > No such file or directory > > > > The only way tomake it go away is to run a make clean in this directory. > Note that crypto is the only test that does that. > > > > To reproduce: > > $ git checkout e3d1361c157b991ea9844b23c211c82e5c9b8408 > > # Probably works on all commits since test changed to libs but I only > checked the procedure on this one) > > $ ./bootstrap && ./configure --enable-cunit-support > --disable-debug-print --enable-test-vald --enable-test-cpp > --enable-test-helper --enable-test-perf && make -j8 > > $ touch test/validation/crypto/crypto.c > > $ ./bootstrap && ./configure --enable-cunit-support > --disable-debug-print --enable-test-vald --enable-test-cpp > --enable-test-helper --enable-test-perf && make -j8 > > > > I tried to see where that comes from but got lost in libtool traces. > > My guess is something get messed up between ODP libcrypto and the system > libcrypto (it's the only test that has the same name as a system library). > Renaming the libcrypto to libcrypto_odp in the Makefile.am seemed to fix > the issue. > > > > If there are any libtool/Automake expert out there, I'm open to a > cleaner fix ! > > > > Nicolas > > I think you're right that the issue is a name collision between this > library and openssl's libcrypto. > > Renaming this library seems like the best solution as creating our own > library named libcrypto that links against openssl's libcrypto is a bad > idea, even if it could be mangled to work. > > This would break our agreed naming though [1] so maybe we should change > that to include the "odp" namespace, e.g. libodp<module>.la > > -- > Stuart. > > [1] > https://git.linaro.org/lng/odp.git/blob/HEAD:/doc/implementers-guide/implementers-guide.adoc#l46 > _______________________________________________ > lng-odp mailing list > [email protected] > https://lists.linaro.org/mailman/listinfo/lng-odp > -- Mike Holmes Technical Manager - Linaro Networking Group Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
_______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
