[oe] [meta-oe][PATCH] poco: update to 1.8.1

[Martin Jansa]

it installs the libCppUnit files like this:

./packages-split/poco-cppunit/usr/lib/libCppUnit.so
./packages-split/poco-dbg/usr/lib/.debug/libCppUnit.so.1
./packages-split/poco-dbg/usr/lib/.debug/libCppUnit.so
./packages-split/poco/usr/lib/libCppUnit.so.1

libCppUnit.so isn't symlink:
-rwxr-xr-x 2 mjansa mjansa 58K Jan 30 16:00 ./packages-split/poco-cppunit/
usr/lib/libCppUnit.so
-rwxr-xr-x 2 mjansa mjansa 58K Jan 30 16:00 ./packages-split/poco/usr/lib/
libCppUnit.so.1

Nor hardlink:
41307884 -rwxr-xr-x 2 mjansa mjansa  58K Jan 30 16:00
./packages-split/poco-cppunit/usr/lib/libCppUnit.so
41307844 -rwxr-xr-x 2 mjansa mjansa 718K Jan 30 15:59
./packages-split/poco-dbg/usr/lib/.debug/libCppUnit.so
41307840 -rwxr-xr-x 2 mjansa mjansa 718K Jan 30 16:00
./packages-split/poco-dbg/usr/lib/.debug/libCppUnit.so.1
41307933 -rwxr-xr-x 2 mjansa mjansa  58K Jan 30 16:00
./packages-split/poco/usr/lib/libCppUnit.so.1

only the debug files are identical:
md5sum ./packages-split/poco-cppunit/usr/lib/libCppUnit.so
./packages-split/poco/usr/lib/libCppUnit.so.1
./packages-split/poco-dbg/usr/lib/.debug/libCppUnit.so.1
./packages-split/poco-dbg/usr/lib/.debug/libCppUnit.so
f91541e8c6f357b827bda035ef5ab0fe  ./packages-split/poco-cppunit/
usr/lib/libCppUnit.so
2ae9c17659a9d903c0343554675f33e6  ./packages-split/poco/usr/lib/
libCppUnit.so.1
f5d6ca0013c9afe592e82d0f929ea0f1  ./packages-split/poco-dbg/usr/
lib/.debug/libCppUnit.so.1
f5d6ca0013c9afe592e82d0f929ea0f1  ./packages-split/poco-dbg/usr/
lib/.debug/libCppUnit.so

So first do_install_ptest should be fixed to preserve libCppUnit.so as a
symlink, then libCppUnit.so should be included in:
FILES_${PN}-dev = "${includedir} ${libdir}/libPoco*.so ${libdir}/cmake"

and shouldn't be included in:
FILES_${PN}-cppunit += "${libdir}/libCppUnit.so*"

You should take advantage of
SOLIBS = ".so.*"
SOLIBSDEV = ".so"
variables from bitbake.conf instead of hardcoding .so and .so.* in FILES
variables.

And poco-cppunit should be added to PACKAGES before PN, now the order
doesn't look right:
#   "${PN}-dbg ${POCO_PACKAGES} ${PN} ${PN}-dev ${PN}-staticdev
${PN}-cppunit"
PACKAGES="poco-dbg poco-xml poco-json poco-mongodb poco-pdf poco-util
poco-net poco-netssl poco-crypto poco-data poco-datasqlite poco-zip
poco-foundation poco poco-dev poco-staticdev poco-cppunit"

Maybe whole packaging should be reworked to use the standard package
definitions and do_split_packages instead of that anonymous function.

Regards,






On Tue, Jan 30, 2018 at 4:07 PM, Martin Jansa <martin.jansa at gmail.com>
wrote:

> Anyone else seeing strange behavior with poco and libCppUnit.so.1 library?
>
> WARNING: poco-1.8.0.1-r0 do_package: poco-cppunit-1.8.0.1 was registered
> as shlib provider for libCppUnit.so.1, changing it to poco-1.8.0.1 because
> it was built later
> WARNING: poco-1.8.0.1-r0 do_package: poco-1.8.0.1 was registered as shlib
> provider for libCppUnit.so.1, changing it to poco-cppunit-1.8.0.1 because
> it was built later
>
> this is happening with the 1.8.0.1 version already, but it would be good
> to fix if it isn't fixed in 1.8.1 already. Maybe just passing explicit
> -DENABLE_TESTS=OFF in
> ${@bb.utils.contains('PTEST_ENABLED', '1', '-DENABLE_TESTS=ON ', '', d)}"
> might help (I haven't tried, I don't use poco anywhere, so I've just
> blacklisted it locally).
>
> On Mon, Jan 22, 2018 at 5:05 PM, akuster808 <akuster808 at gmail.com> wrote:
>
>>
>>
>> On 01/17/2018 06:42 AM, Pascal Bach wrote:
>> > Signed-off-by: Pascal Bach <pascal.bach at siemens.com>
>>
>> Failed to build. Please see
>>
>> http://errors.yoctoproject.org/Errors/Build/51576/
>> > ---
>> >  meta-oe/recipes-support/poco/{poco_1.8.0.1.bb => poco_1.8.1.bb} | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >  rename meta-oe/recipes-support/poco/{poco_1.8.0.1.bb => poco_1.8.1.bb}
>> (98%)
>> >
>> > diff --git a/meta-oe/recipes-support/poco/poco_1.8.0.1.bb
>> b/meta-oe/recipes-support/poco/poco_1.8.1.bb
>> > similarity index 98%
>> > rename from meta-oe/recipes-support/poco/poco_1.8.0.1.bb
>> > rename to meta-oe/recipes-support/poco/poco_1.8.1.bb
>> > index 160b4d391..e44ff4470 100644
>> > --- a/meta-oe/recipes-support/poco/poco_1.8.0.1.bb
>> > +++ b/meta-oe/recipes-support/poco/poco_1.8.1.bb
>> > @@ -12,7 +12,7 @@ inherit cmake ptest
>> >
>> >  BBCLASSEXTEND = "native"
>> >
>> > -SRCREV = "af527ab21fca5ab2659285408aec9920ed7c7b17"
>> > +SRCREV = "2eca6c1813632fb779e07f1e1968afefa81e1dd0"
>> >  SRC_URI = " \
>> >      git://github.com/pocoproject/poco.git \
>> >      file://run-ptest \
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel at lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>>
>
>