On Fri, Jun 26, 2020 at 5:47 AM Alexander Kanavin
<[email protected]> wrote:
>
> Can you link to the failures please?

Yes, ppc failed too.

https://errors.yoctoproject.org/Errors/Details/417489/
https://errors.yoctoproject.org/Errors/Details/417486/
https://errors.yoctoproject.org/Errors/Details/417484/

Thanks for taking a look!

Steve
>
> On Fri 26. Jun 2020 at 17.27, Steve Sakoman <[email protected]> wrote:
>>
>> On Thu, Jun 25, 2020 at 6:23 AM Steve Sakoman <[email protected]> wrote:
>> >
>> > On Thu, Jun 25, 2020 at 6:09 AM Alexander Kanavin
>> > <[email protected]> wrote:
>> > >
>> > > I think you may have to recreate them by hand.
>> >
>> > OK, I'll put together a patch for the 66.1 recipe in dunfell that
>> > implements the changes you have in the second patch of the series.  In
>> > the interest of minimizing changes I'll keep the .inc file separate as
>> > is.
>>
>> I recreated the patch for 66.1 and did a couple of quick local builds
>> and things looked fine.
>>
>> On the autobuilder I get failures for edgerouter and mips.  I seem to
>> recall some discussion about this (perhaps on irc?) but can only find
>> one thread here on the list back when Khem's original patch went in.
>>
>> Did you recall any issues like this?
>>
>> Steve
>>
>>
>> > > On Thu, 25 Jun 2020 at 18:04, Steve Sakoman <[email protected]> wrote:
>> > >>
>> > >> Is this patch series suitable for dunfell as-is (the concern being the
>> > >> contents of the version bump) or would a backport to 66.1 be
>> > >> necessary?
>> > >>
>> > >> Steve
>> > >>
>> > >> On Tue, Jun 23, 2020 at 11:09 AM Alexander Kanavin
>> > >> <[email protected]> wrote:
>> > >> >
>> > >> > Both parts are rather convoluted, so it is better to have them in a 
>> > >> > single file.
>> > >> >
>> > >> > Signed-off-by: Alexander Kanavin <[email protected]>
>> > >> > ---
>> > >> >  meta/recipes-support/icu/icu.inc     | 84 ---------------------------
>> > >> >  meta/recipes-support/icu/icu_67.1.bb | 85 
>> > >> > +++++++++++++++++++++++++++-
>> > >> >  2 files changed, 84 insertions(+), 85 deletions(-)
>> > >> >  delete mode 100644 meta/recipes-support/icu/icu.inc
>> > >> >
>> > >> > diff --git a/meta/recipes-support/icu/icu.inc 
>> > >> > b/meta/recipes-support/icu/icu.inc
>> > >> > deleted file mode 100644
>> > >> > index 6d9728903d..0000000000
>> > >> > --- a/meta/recipes-support/icu/icu.inc
>> > >> > +++ /dev/null
>> > >> > @@ -1,84 +0,0 @@
>> > >> > -SUMMARY = "International Component for Unicode libraries"
>> > >> > -DESCRIPTION = "The International Component for Unicode (ICU) is a 
>> > >> > mature, \
>> > >> > -portable set of C/C++ and Java libraries for Unicode support, 
>> > >> > software \
>> > >> > -internationalization (I18N) and globalization (G11N), giving 
>> > >> > applications the \
>> > >> > -same results on all platforms."
>> > >> > -HOMEPAGE = "http://site.icu-project.org/";
>> > >> > -
>> > >> > -LICENSE = "ICU"
>> > >> > -DEPENDS = "icu-native"
>> > >> > -DEPENDS_class-native = ""
>> > >> > -
>> > >> > -CVE_PRODUCT = "international_components_for_unicode"
>> > >> > -
>> > >> > -S = "${WORKDIR}/icu/source"
>> > >> > -SPDX_S = "${WORKDIR}/icu"
>> > >> > -STAGING_ICU_DIR_NATIVE = "${STAGING_DATADIR_NATIVE}/${BPN}/${PV}"
>> > >> > -
>> > >> > -BINCONFIG = "${bindir}/icu-config"
>> > >> > -
>> > >> > -ICU_MAJOR_VER = "${@d.getVar('PV').split('.')[0]}"
>> > >> > -
>> > >> > -inherit autotools pkgconfig binconfig multilib_script
>> > >> > -
>> > >> > -MULTILIB_SCRIPTS = "${PN}-dev:${bindir}/icu-config"
>> > >> > -
>> > >> > -# ICU needs the native build directory as an argument to its 
>> > >> > --with-cross-build option when
>> > >> > -# cross-compiling. Taken the situation that different builds may 
>> > >> > share a common sstate-cache
>> > >> > -# into consideration, the native build directory needs to be staged.
>> > >> > -EXTRA_OECONF = "--with-cross-build=${STAGING_ICU_DIR_NATIVE}"
>> > >> > -EXTRA_OECONF_class-native = ""
>> > >> > -EXTRA_OECONF_class-nativesdk = 
>> > >> > "--with-cross-build=${STAGING_ICU_DIR_NATIVE}"
>> > >> > -
>> > >> > -EXTRA_OECONF_append_class-target = 
>> > >> > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' 
>> > >> > --with-data-packaging=archive', '', d)}"
>> > >> > -TARGET_CXXFLAGS_append = 
>> > >> > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' 
>> > >> > -DICU_DATA_DIR=\\""${datadir}/${BPN}/${PV}\\""', '', d)}"
>> > >> > -
>> > >> > -ASNEEDED = ""
>> > >> > -
>> > >> > -do_compile_prepend_class-target () {
>> > >> > -       # Make sure certain build host references do not end up being 
>> > >> > compiled
>> > >> > -       # in the image. This only affects libicutu and icu-dbg
>> > >> > -       sed  \
>> > >> > -           -e 's,DU_BUILD=,DU_BUILD_unused=,g' \
>> > >> > -           -e '/^CPPFLAGS.*/ s,--sysroot=${STAGING_DIR_TARGET},,g' \
>> > >> > -           -i ${B}/tools/toolutil/Makefile
>> > >> > -}
>> > >> > -
>> > >> > -PREPROCESS_RELOCATE_DIRS = "${datadir}/${BPN}/${PV}"
>> > >> > -do_install_append_class-native() {
>> > >> > -       mkdir -p ${D}/${STAGING_ICU_DIR_NATIVE}/config
>> > >> > -       cp -r ${B}/config/icucross.mk 
>> > >> > ${D}/${STAGING_ICU_DIR_NATIVE}/config
>> > >> > -       cp -r ${B}/config/icucross.inc 
>> > >> > ${D}/${STAGING_ICU_DIR_NATIVE}/config
>> > >> > -       cp -r ${B}/lib ${D}/${STAGING_ICU_DIR_NATIVE}
>> > >> > -       cp -r ${B}/bin ${D}/${STAGING_ICU_DIR_NATIVE}
>> > >> > -       cp -r ${B}/tools ${D}/${STAGING_ICU_DIR_NATIVE}
>> > >> > -}
>> > >> > -
>> > >> > -do_install_append_class-target() {
>> > >> > -    # The native pkgdata can not generate the correct data file.
>> > >> > -    # Use icupkg to re-generate it.
>> > >> > -    if [ "${SITEINFO_ENDIANNESS}" = "be" ] ; then
>> > >> > -        rm -f ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat
>> > >> > -        icupkg -tb ${S}/data/in/icudt${ICU_MAJOR_VER}l.dat 
>> > >> > ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat
>> > >> > -    fi
>> > >> > -
>> > >> > -       # Remove build host references...
>> > >> > -       sed -i  \
>> > >> > -           -e 's,--sysroot=${STAGING_DIR_TARGET},,g' \
>> > >> > -           -e 's|${DEBUG_PREFIX_MAP}||g' \
>> > >> > -           -e 's:${HOSTTOOLS_DIR}/::g' \
>> > >> > -           ${D}/${bindir}/icu-config 
>> > >> > ${D}/${libdir}/${BPN}/${PV}/Makefile.inc \
>> > >> > -           ${D}/${libdir}/${BPN}/${PV}/pkgdata.inc
>> > >> > -}
>> > >> > -
>> > >> > -PACKAGES =+ "libicudata libicuuc libicui18n libicutu libicuio"
>> > >> > -
>> > >> > -FILES_${PN}-dev += "${libdir}/${BPN}/"
>> > >> > -
>> > >> > -FILES_libicudata = "${libdir}/libicudata.so.*"
>> > >> > -FILES_libicuuc = "${libdir}/libicuuc.so.*"
>> > >> > -FILES_libicui18n = "${libdir}/libicui18n.so.*"
>> > >> > -FILES_libicutu = "${libdir}/libicutu.so.*"
>> > >> > -FILES_libicuio = "${libdir}/libicuio.so.*"
>> > >> > -
>> > >> > -BBCLASSEXTEND = "native nativesdk"
>> > >> > diff --git a/meta/recipes-support/icu/icu_67.1.bb 
>> > >> > b/meta/recipes-support/icu/icu_67.1.bb
>> > >> > index 58638507c9..5814fa41bd 100644
>> > >> > --- a/meta/recipes-support/icu/icu_67.1.bb
>> > >> > +++ b/meta/recipes-support/icu/icu_67.1.bb
>> > >> > @@ -1,4 +1,87 @@
>> > >> > -require icu.inc
>> > >> > +SUMMARY = "International Component for Unicode libraries"
>> > >> > +DESCRIPTION = "The International Component for Unicode (ICU) is a 
>> > >> > mature, \
>> > >> > +portable set of C/C++ and Java libraries for Unicode support, 
>> > >> > software \
>> > >> > +internationalization (I18N) and globalization (G11N), giving 
>> > >> > applications the \
>> > >> > +same results on all platforms."
>> > >> > +HOMEPAGE = "http://site.icu-project.org/";
>> > >> > +
>> > >> > +LICENSE = "ICU"
>> > >> > +DEPENDS = "icu-native"
>> > >> > +DEPENDS_class-native = ""
>> > >> > +
>> > >> > +CVE_PRODUCT = "international_components_for_unicode"
>> > >> > +
>> > >> > +S = "${WORKDIR}/icu/source"
>> > >> > +SPDX_S = "${WORKDIR}/icu"
>> > >> > +STAGING_ICU_DIR_NATIVE = "${STAGING_DATADIR_NATIVE}/${BPN}/${PV}"
>> > >> > +
>> > >> > +BINCONFIG = "${bindir}/icu-config"
>> > >> > +
>> > >> > +ICU_MAJOR_VER = "${@d.getVar('PV').split('.')[0]}"
>> > >> > +
>> > >> > +inherit autotools pkgconfig binconfig multilib_script
>> > >> > +
>> > >> > +MULTILIB_SCRIPTS = "${PN}-dev:${bindir}/icu-config"
>> > >> > +
>> > >> > +# ICU needs the native build directory as an argument to its 
>> > >> > --with-cross-build option when
>> > >> > +# cross-compiling. Taken the situation that different builds may 
>> > >> > share a common sstate-cache
>> > >> > +# into consideration, the native build directory needs to be staged.
>> > >> > +EXTRA_OECONF = "--with-cross-build=${STAGING_ICU_DIR_NATIVE}"
>> > >> > +EXTRA_OECONF_class-native = ""
>> > >> > +EXTRA_OECONF_class-nativesdk = 
>> > >> > "--with-cross-build=${STAGING_ICU_DIR_NATIVE}"
>> > >> > +
>> > >> > +EXTRA_OECONF_append_class-target = 
>> > >> > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' 
>> > >> > --with-data-packaging=archive', '', d)}"
>> > >> > +TARGET_CXXFLAGS_append = 
>> > >> > "${@oe.utils.conditional('SITEINFO_ENDIANNESS', 'be', ' 
>> > >> > -DICU_DATA_DIR=\\""${datadir}/${BPN}/${PV}\\""', '', d)}"
>> > >> > +
>> > >> > +ASNEEDED = ""
>> > >> > +
>> > >> > +do_compile_prepend_class-target () {
>> > >> > +       # Make sure certain build host references do not end up being 
>> > >> > compiled
>> > >> > +       # in the image. This only affects libicutu and icu-dbg
>> > >> > +       sed  \
>> > >> > +           -e 's,DU_BUILD=,DU_BUILD_unused=,g' \
>> > >> > +           -e '/^CPPFLAGS.*/ s,--sysroot=${STAGING_DIR_TARGET},,g' \
>> > >> > +           -i ${B}/tools/toolutil/Makefile
>> > >> > +}
>> > >> > +
>> > >> > +PREPROCESS_RELOCATE_DIRS = "${datadir}/${BPN}/${PV}"
>> > >> > +do_install_append_class-native() {
>> > >> > +       mkdir -p ${D}/${STAGING_ICU_DIR_NATIVE}/config
>> > >> > +       cp -r ${B}/config/icucross.mk 
>> > >> > ${D}/${STAGING_ICU_DIR_NATIVE}/config
>> > >> > +       cp -r ${B}/config/icucross.inc 
>> > >> > ${D}/${STAGING_ICU_DIR_NATIVE}/config
>> > >> > +       cp -r ${B}/lib ${D}/${STAGING_ICU_DIR_NATIVE}
>> > >> > +       cp -r ${B}/bin ${D}/${STAGING_ICU_DIR_NATIVE}
>> > >> > +       cp -r ${B}/tools ${D}/${STAGING_ICU_DIR_NATIVE}
>> > >> > +}
>> > >> > +
>> > >> > +do_install_append_class-target() {
>> > >> > +    # The native pkgdata can not generate the correct data file.
>> > >> > +    # Use icupkg to re-generate it.
>> > >> > +    if [ "${SITEINFO_ENDIANNESS}" = "be" ] ; then
>> > >> > +        rm -f ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat
>> > >> > +        icupkg -tb ${S}/data/in/icudt${ICU_MAJOR_VER}l.dat 
>> > >> > ${D}/${datadir}/${BPN}/${PV}/icudt${ICU_MAJOR_VER}b.dat
>> > >> > +    fi
>> > >> > +
>> > >> > +       # Remove build host references...
>> > >> > +       sed -i  \
>> > >> > +           -e 's,--sysroot=${STAGING_DIR_TARGET},,g' \
>> > >> > +           -e 's|${DEBUG_PREFIX_MAP}||g' \
>> > >> > +           -e 's:${HOSTTOOLS_DIR}/::g' \
>> > >> > +           ${D}/${bindir}/icu-config 
>> > >> > ${D}/${libdir}/${BPN}/${PV}/Makefile.inc \
>> > >> > +           ${D}/${libdir}/${BPN}/${PV}/pkgdata.inc
>> > >> > +}
>> > >> > +
>> > >> > +PACKAGES =+ "libicudata libicuuc libicui18n libicutu libicuio"
>> > >> > +
>> > >> > +FILES_${PN}-dev += "${libdir}/${BPN}/"
>> > >> > +
>> > >> > +FILES_libicudata = "${libdir}/libicudata.so.*"
>> > >> > +FILES_libicuuc = "${libdir}/libicuuc.so.*"
>> > >> > +FILES_libicui18n = "${libdir}/libicui18n.so.*"
>> > >> > +FILES_libicutu = "${libdir}/libicutu.so.*"
>> > >> > +FILES_libicuio = "${libdir}/libicuio.so.*"
>> > >> > +
>> > >> > +BBCLASSEXTEND = "native nativesdk"
>> > >> >
>> > >> >  LIC_FILES_CHKSUM = 
>> > >> > "file://../LICENSE;md5=a3808a5b70071b07f87ff2205e4d75a0"
>> > >> >
>> > >> > --
>> > >> > 2.27.0
>> > >> >
>> > >> > 
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#140023): 
https://lists.openembedded.org/g/openembedded-core/message/140023
Mute This Topic: https://lists.openembedded.org/mt/75069295/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to